Skip to content

(2.12) [FIXED] Required API level error spam#7711

Merged
neilalexander merged 1 commit intomainfrom
maurice/req-api-lvl-err-spam
Jan 9, 2026
Merged

(2.12) [FIXED] Required API level error spam#7711
neilalexander merged 1 commit intomainfrom
maurice/req-api-lvl-err-spam

Conversation

@MauriceVanVeen
Copy link
Copy Markdown
Member

The required API level error (when used) must only be sent AFTER checking we're meta/stream leader. Otherwise multiple servers could trigger and would essentially spam this error response back to the client.

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

@MauriceVanVeen MauriceVanVeen requested a review from a team as a code owner January 9, 2026 13:29
Copy link
Copy Markdown
Member

@neilalexander neilalexander left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Maurice van Veen <github@mauricevanveen.com>
@MauriceVanVeen MauriceVanVeen force-pushed the maurice/req-api-lvl-err-spam branch from 2457e21 to 99afec3 Compare January 9, 2026 13:50
@neilalexander neilalexander merged commit 2ed9d69 into main Jan 9, 2026
68 of 70 checks passed
@neilalexander neilalexander deleted the maurice/req-api-lvl-err-spam branch January 9, 2026 14:16
neilalexander added a commit that referenced this pull request Jan 15, 2026
Includes the following:

- #7704
- #7708
- #7710
- #7711
- #7709
- #7712
- #7716
- #7720
- #7721
- #7723
- #7728

Signed-off-by: Neil Twigg <neil@nats.io>
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