Skip to content

Return 400 for unknown /connz query parameters#8048

Open
shiroonigami23-ui wants to merge 1 commit into
nats-io:mainfrom
shiroonigami23-ui:fix-704-connz-unknown-query-params
Open

Return 400 for unknown /connz query parameters#8048
shiroonigami23-ui wants to merge 1 commit into
nats-io:mainfrom
shiroonigami23-ui:fix-704-connz-unknown-query-params

Conversation

@shiroonigami23-ui
Copy link
Copy Markdown

Fixes #704.

Summary

  • validate query params for /connz
  • return 400 Bad Request for unknown parameters
  • keep callback allowed for JSONP support
  • add regression test for /connz?testpub=foo

Verification

  • go test ./server -run ^(TestMonitorConnzBadParams|TestMonitorConnz)$ -count=1

@shiroonigami23-ui shiroonigami23-ui requested a review from a team as a code owner April 19, 2026 20:39
Signed-off-by: Shiro Oni <shiroonigami23@gmail.com>
@shiroonigami23-ui shiroonigami23-ui force-pushed the fix-704-connz-unknown-query-params branch from 46a87a6 to ab72d7d Compare April 21, 2026 20:20
@shiroonigami23-ui
Copy link
Copy Markdown
Author

Updated: I amended the commit with a proper DCO sign-off (Signed-off-by: Shiro Oni shiroonigami23@gmail.com) and force-pushed the branch.

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.

Monitoring ignores invalid args

1 participant