Update deposit-service OpenAPI spec#109
Conversation
![rhinestone-kevin[bot]](https://avatars.githubusercontent.com/u/127950714?s=60&v=4){kind=small}
greg-rhinestone
left a comment
greg-rhinestone
left a comment
There was a problem hiding this comment.
🤖 Greg · ✅ LGTM
Regenerates the deposit-service spec from rhinestonewtf/deposit-service-processor@690e44e, adding deposits-scope 403 responses across the affected account/deposit endpoints. This matches the upstream key-scope enforcement surface.
Risk: Low — generated spec-only change documenting additive auth errors
Destiner
force-pushed
the
update/deposit-processor
branch
from
019bee9 to
a7cbe81
Compare
Destiner
force-pushed
the
update/deposit-processor
branch
from
a7cbe81 to
410e99f
Compare
greg-rhinestone
left a comment
greg-rhinestone
left a comment
There was a problem hiding this comment.
🤖 Greg · 🔄 Re-review · ✅ LGTM
No effective deposit-service.json change since the last Greg review; the current generated spec still contains the already-reviewed deposits-scope 403 response coverage.
Destiner
force-pushed
the
update/deposit-processor
branch
from
410e99f to
6337692
Compare
greg-rhinestone
left a comment
greg-rhinestone
left a comment
There was a problem hiding this comment.
🤖 Greg · 🔄 Re-review · ✅ LGTM
Generated deposit-service spec now includes the deposits-scope 403 response on the remaining account, deposit, withdrawal, and webhook endpoints from rhinestonewtf/deposit-service-processor@6f83c1a. No new compatibility issue surfaced.
Destiner
force-pushed
the
update/deposit-processor
branch
2 times, most recently
from
a2189ba to
10c746c
Compare
…t-service-processor@a5b8605223b23e08b9858a423ec4f12c4fd2ce1a
Destiner
force-pushed
the
update/deposit-processor
branch
from
10c746c to
b4ebe5b
Compare
2 participants
This PR updates the OpenAPI spec generated from rhinestonewtf/deposit-service-processor@a5b8605223b23e08b9858a423ec4f12c4fd2ce1a.