Skip to content

Fix/inactive streams#987

Closed
TheShnider wants to merge 3 commits into
LabsCrypt:mainfrom
TheShnider:fix/inactive-streams
Closed

Fix/inactive streams#987
TheShnider wants to merge 3 commits into
LabsCrypt:mainfrom
TheShnider:fix/inactive-streams

Conversation

@TheShnider

Copy link
Copy Markdown

closes #812

@ogazboiz

ogazboiz commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

closing as superseded by #970. you both edited the exact same auth-jwt signature-tamper line (you flip a middle character, #970 replaces the whole signature with an invalid string) and they conflict, so only one can land. #970 is the larger sorobanService change and already makes that test's signature reliably invalid. your middle-char approach was the more surgical one though, good catch on the weak last-char tamper. thanks!

@ogazboiz ogazboiz closed this Jul 3, 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.

[Backend] POST /v1/streams/:streamId/top-up does not reject cancelled/inactive or paused streams

2 participants