Page MenuHomePhorge
Differential D293

Fix process response without json body for GetVersionsAction
ClosedPublic
Actions

Authored by nannanko on Mar 18 2026, 7:31 AM.

Details

Reviewers
tusooa
Group Reviewers
O1: the Kazv Project
Commits
rLe70f5822bcc9: Fix process response without json body for GetVersionsAction
Summary

Kazv::processResponse(ClientModel m, GetVersionsResponse r) cannot process response without json body, this diff fixed it.

Type: fix

Test Plan

Verify unit tests pass.

Diff Detail

Repository
rL libkazv
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

nannanko created this revision.Mar 18 2026, 7:31 AM
Owners added a reviewer: O1: the Kazv Project.Mar 18 2026, 7:31 AM
tusooa published this revision for review.Mar 18 2026, 7:37 AM
tusooa added inline comments.
src/client/actions/auth.cpp
179–181

You don't need to create a new lambda for this. failWithResponse(r) already meets the requirement of an effect.

tusooa requested changes to this revision.Mar 18 2026, 7:37 AM
This revision now requires changes to proceed.Mar 18 2026, 7:37 AM
Harbormaster failed remote builds in B568: Diff 882!Mar 18 2026, 7:45 AM
nannanko updated this revision to Diff 884.Mar 18 2026, 9:01 AM

Use failWithResponse(r) instead of lambda

nannanko marked an inline comment as done.Mar 18 2026, 9:01 AM
Harbormaster failed remote builds in B569: Diff 884!Mar 18 2026, 9:15 AM
tusooa accepted this revision.Mar 19 2026, 8:00 PM

the test failure is not the fault of this diff. the coverage job did pass, the failing is because of lilybuild.

This revision is now accepted and ready to land.Mar 19 2026, 8:00 PM
This revision was landed with ongoing or failed builds.Mar 21 2026, 6:57 AM
Closed by commit rLe70f5822bcc9: Fix process response without json body for GetVersionsAction. · Explain Why
This revision was automatically updated to reflect the committed changes.
nannanko added a commit: rLe70f5822bcc9: Fix process response without json body for GetVersionsAction.

Revision Contents
Changeset List

PathSizePackages
src/
client/
actions/
16 linesO1: the Kazv Project
tests/
client/
10 linesO1: the Kazv Project

Diff 888

View Options

src/client/actions/auth.cpp

Loading...
View Options

src/tests/client/get-versions-test.cpp

Loading...