fix(protocol): align retry_after with the canonical error shape#5954
Conversation
There was a problem hiding this comment.
Argus is not auto-reviewing this PR because it modifies protected paths that require human review (static/schemas/source/enums/error-code.json, static/schemas/source/protocol/get-adcp-capabilities-response.json). A human reviewer should review and merge this PR; Argus will resume on subsequent PRs once these changes land on main.
There was a problem hiding this comment.
Argus is not auto-reviewing this PR because it modifies protected paths that require human review (static/schemas/source/enums/error-code.json, static/schemas/source/protocol/get-adcp-capabilities-response.json). A human reviewer should review and merge this PR; Argus will resume on subsequent PRs once these changes land on main.
|
This PR contains a single change - looks good to me Problem: Our guidance told developers to read the “wait this long before trying again” hint from a different place than where the actual rules said to put it, so anyone following the instructions would look in the wrong spot and miss the hint entirely. |
Summary
error.retry_afternumberschemaWhy
Prose and compliance assets referenced
error.details.retry_after, whilecore/error.jsondefinesretry_afteron the error object itself. The rate-limit harness therefore risked rejecting conformant sellers and rewarding the wrong wire shape.Validation
git diff --checkThe separate number-versus-integer decision is tracked in #5953.
Main-line portion of #5948.