Skip to content

Oauth2: Support variable expansion when checking resource access #13941

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Jun 2, 2025
Merged

Oauth2: Support variable expansion when checking resource access #13941

merged 4 commits into from
Jun 2, 2025

Conversation

@MarcialRosales
Copy link
Contributor

@MarcialRosales MarcialRosales commented May 23, 2025 •
edited
Loading

Proposed Changes

This PR adds variable expansion when checking resource access or vhost access on oauth2 authentication backend. RabbitMQ only supported variable expansion when checking topic access.

It addresses feature request #13894

Doc's PR rabbitmq/rabbitmq-website#2278

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

@MarcialRosales MarcialRosales self-assigned this May 23, 2025
@MarcialRosales MarcialRosales changed the title Support variable expansion in oauth2 resource permission Oauth2: Support variable expansion when checking resource access May 23, 2025
@michaelklishin michaelklishin added this to the 4.2.0 milestone May 23, 2025
@MarcialRosales MarcialRosales force-pushed the feature-13894 branch 2 times, most recently from 3cc7ada to ba3471d Compare May 28, 2025 14:14
@MarcialRosales MarcialRosales mentioned this pull request May 29, 2025
Merged
@MarcialRosales MarcialRosales marked this pull request as ready for review May 30, 2025 13:47
@michaelklishin michaelklishin merged commit e19d2b0 into main Jun 2, 2025
1386 of 1391 checks passed
@michaelklishin michaelklishin deleted the feature-13894 branch June 2, 2025 17:30
@mergify mergify bot mentioned this pull request Jun 2, 2025
Merged
6 tasks
michaelklishin added a commit that referenced this pull request Jun 2, 2025
@michaelklishin
Merge pull request #14008 from rabbitmq/mergify/bp/v4.1.x/pr-13941
3139268 
Oauth2: Support variable expansion when checking resource access (backport #13941)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@michaelklishin michaelklishin michaelklishin approved these changes

Assignees

@MarcialRosales MarcialRosales

Labels

backport-v4.1.x rabbitmq-auth-backend-oauth2

Projects

None yet

Milestone

4.2.0

Development

Successfully merging this pull request may close these issues.

3 participants

@MarcialRosales @michaelklishin