Skip to content

andromeda - #368

Open
almeida378lops wants to merge 1 commit into
Conway-Research:mainfrom
almeida378lops:main
Open

andromeda#368
almeida378lops wants to merge 1 commit into
Conway-Research:mainfrom
almeida378lops:main

Conversation

@almeida378lops

Copy link
Copy Markdown

No description provided.

The SIWE nonce is single-use; the server consumes it on the first /verify
attempt. Because /verify runs through ResilientHttpClient with retries and
5xx is retryable, a server-side failure (observed: 500 "Database error"
while persisting the session) triggers a retry that reuses the spent nonce
and comes back as a misleading "401 Invalid or expired nonce". The true
error is lost and provisioning looks like a client/nonce problem.

Pass retries: 0 on the /verify call so the real verification error surfaces
to the caller instead of being masked by a retry.

Adds a ResilientHttpClient test that a retryable 5xx with retries:0 is
returned directly without retrying.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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.

2 participants