Skip to content

[WALLET-184] Add response_uri to DC API request - #35

Merged
frostevent merged 1 commit into
mainfrom
jeremie/wallet-184
Jul 31, 2026
Merged

[WALLET-184] Add response_uri to DC API request#35
frostevent merged 1 commit into
mainfrom
jeremie/wallet-184

Conversation

@frostevent

Copy link
Copy Markdown
Collaborator

Summary

Adds two fields to the signed DC API request path in packages/server:

  • response_uri — added to the DCAPIAuthorizationRequest payload, populated from the client's configured callbackUri.
  • expected_origins — a new required expectedOrigins parameter on signedDcApiRequest, typed as a non-empty array ([string, ...string[]]) and validated at runtime (rejects an empty array). It is emitted as expected_origins in the signed request payload.

signedDcApiRequest is now async so the empty-array validation surfaces as a promise rejection, consistent with the other config-validation errors.

Tests

  • Existing signedDcApiRequest test updated to pass expectedOrigins and assert response_uri / expected_origins in the signed payload.
  • New test covering rejection of an empty expectedOrigins array.
  • Full suite passes; yarn check-all (format, lint, build, publint) is green.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@frostevent
frostevent enabled auto-merge (squash) July 31, 2026 10:13
@frostevent
frostevent disabled auto-merge July 31, 2026 10:15
@frostevent
frostevent merged commit 34d2e1e into main Jul 31, 2026
10 checks passed
@frostevent
frostevent deleted the jeremie/wallet-184 branch July 31, 2026 10:15
@frostevent frostevent mentioned this pull request Jul 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant