Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cherry picks for v2.10.20 #5843

Merged
merged 3 commits into from
Aug 29, 2024
Merged

Cherry picks for v2.10.20 #5843

merged 3 commits into from
Aug 29, 2024

Conversation

wallyqs
Copy link
Member

@wallyqs wallyqs commented Aug 28, 2024

A regression was introduced in
#5821 where CAS operations on
a R1 stream would succeed even if they should be rejected. Whereas on
R3/clustered they would be rejected.

Resolves: #5840

Signed-off-by: Maurice van Veen <github@mauricevanveen.com>

---------

Signed-off-by: Waldemar Quevedo <wally@nats.io>
Signed-off-by: Maurice van Veen <github@mauricevanveen.com>
Co-authored-by: Waldemar Quevedo <wally@nats.io>
@wallyqs wallyqs requested a review from a team as a code owner August 28, 2024 18:31
Reported-by: Trail of Bits <https://www.trailofbits.com>
Signed-off-by: Neil Twigg <neil@nats.io>
Prevents potential re-tagging attacks on third-party workflows.

Reported-by: Trail of Bits <https://www.trailofbits.com>
Signed-off-by: Neil Twigg <neil@nats.io>
Copy link
Member

@derekcollison derekcollison left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wallyqs wallyqs changed the title Cherry picks for v2.10.20 RC1 Cherry picks for v2.10.20 Aug 29, 2024
@wallyqs wallyqs merged commit a24fe36 into release/v2.10.20 Aug 29, 2024
5 checks passed
@wallyqs wallyqs deleted the downstream/v2.10.20 branch August 29, 2024 17:38
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.

4 participants