Skip to content

Commit 511de32

Browse files
chrismaz11claude
andcommitted
fix(revoke): add status field to revoke response for smoke test
POST /api/v1/receipt/:receiptId/revoke was returning { result: 'REVOKED' } but v2-integration.test.ts asserts .status. Added status: 'REVOKED' to both response paths (fresh revoke and already-revoked) for consistency. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 7d54a0d commit 511de32

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

apps/api/src/server.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1960,6 +1960,7 @@ export async function buildServer(options: BuildServerOptions = {}) {
19601960

19611961
if (record.revoked) {
19621962
return reply.send({
1963+
status: 'REVOKED',
19631964
receiptStatus: 'revoked' satisfies ExternalReceiptStatus,
19641965
result: 'ALREADY_REVOKED'
19651966
});
@@ -1982,6 +1983,7 @@ export async function buildServer(options: BuildServerOptions = {}) {
19821983
);
19831984

19841985
return reply.send({
1986+
status: 'REVOKED',
19851987
receiptStatus: 'revoked' satisfies ExternalReceiptStatus,
19861988
result: 'REVOKED',
19871989
issuerId: revocationVerification.issuerId

0 commit comments

Comments
 (0)