Skip to content

Refactor HTTP API responses (backport #16709) (backport #16742)#16746

Merged
michaelklishin merged 13 commits into
v4.2.xfrom
mergify/bp/v4.2.x/pr-16742
Jun 20, 2026
Merged

Refactor HTTP API responses (backport #16709) (backport #16742)#16746
michaelklishin merged 13 commits into
v4.2.xfrom
mergify/bp/v4.2.x/pr-16742

Conversation

@mergify

@mergify mergify Bot commented Jun 19, 2026

Copy link
Copy Markdown

Proposed Changes

Favor 404 over 401 when user has no permission to a vhost
Support float as token expiry data type not only integer
Enforce is same user or admin/monitor when checking for user-limits for a specific user.
In rabbitmq_top plugin, validate sort parameters against a fix set of values

Types of Changes

What types of changes does your code introduce to this project?
Put an x in the boxes that apply

  • Bug fix (non-breaking change which fixes issue #NNNN)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause an observable behavior change in existing systems)
  • Documentation improvements (corrections, new content, etc)
  • Cosmetic change (whitespace, formatting, etc)
  • Build system and/or CI
This is an automatic backport of pull request #16709 done by [Mergify](https://mergify.com).
This is an automatic backport of pull request #16742 done by [Mergify](https://mergify.com).

@MarcialRosales @mergify
Use 404 istead of 401 when not authorized
fec406f 
(cherry picked from commit b11526f)
(cherry picked from commit 1a868b5)
@MarcialRosales @mergify
Accept float also for token expiry
c195102 
(cherry picked from commit 0c21c93)
(cherry picked from commit c15ce93)
@MarcialRosales @mergify
Check user or admin/monitor for user-limits
7af6d5a 
(cherry picked from commit 1e18471)
(cherry picked from commit 1010efa)
@MarcialRosales @mergify
Fix failing test cases
e4061a9 
(cherry picked from commit f318602)
(cherry picked from commit 4e47109)
@MarcialRosales @mergify
Fix : Administrators continue to get the full
39ccffc 
(cherry picked from commit b288d41)
(cherry picked from commit 32d189e)
@MarcialRosales @mergify
Fix issue with cookie path for oauth2 flows
0b6bfcf 
(cherry picked from commit 9892338)
(cherry picked from commit 4fd281c)
@MarcialRosales @mergify
Validate sort keys against fixed set of allowed values
14cea73 
(cherry picked from commit 7a86828)
(cherry picked from commit f491f68)
@MarcialRosales @mergify
Update tests to check against 404 rather than 401
2620861 
when the user is not authorized to access teh vhost

(cherry picked from commit 9c5b68f)
(cherry picked from commit 47f99b4)
@MarcialRosales @mergify
Add a fake compute time when authenticating non-existing user
e6579cc 
(cherry picked from commit 29eb012)
(cherry picked from commit 400dbf8)
@MarcialRosales @mergify
Check permission to delete partition before
541d6f2 
deleting the superstream

(cherry picked from commit cc690f1)
(cherry picked from commit 03afcea)
@MarcialRosales @mergify
Fix indentation
1f594ba 
(cherry picked from commit 2a47978)
(cherry picked from commit b87d3da)
@MarcialRosales @mergify
Use a better design for adding a prefix to paths
4d5cd5b 
(cherry picked from commit 208afc1)
(cherry picked from commit 6c7531f)
@michaelklishin michaelklishin added this to the 4.2.9 milestone Jun 19, 2026
@mergify

mergify Bot commented Jun 19, 2026

Copy link
Copy Markdown
Author

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@michaelklishin

michaelklishin commented Jun 20, 2026

Copy link
Copy Markdown
Collaborator

#16747 will fix the management plugin failures specific to v4.2.x.

@michaelklishin
michaelklishin merged commit aedc6f4 into v4.2.x Jun 20, 2026
591 of 595 checks passed
@michaelklishin
michaelklishin deleted the mergify/bp/v4.2.x/pr-16742 branch June 20, 2026 03:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

4.2.9

Development

Successfully merging this pull request may close these issues.

2 participants

@michaelklishin @MarcialRosales