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.

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

Support keycloak custom format via configuration #13216

Merged
merged 6 commits into from
Feb 13, 2025
Merged

Support keycloak custom format via configuration #13216

merged 6 commits into from
Feb 13, 2025

Conversation

MarcialRosales
Copy link
Contributor

@MarcialRosales MarcialRosales commented Feb 7, 2025 •
edited
Loading

Proposed Changes

Implements feature #12324

Docs pr:

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 Feb 7, 2025
@mergify mergify bot added the bazel label Feb 7, 2025
@MarcialRosales MarcialRosales mentioned this pull request Feb 7, 2025
Merged
@MarcialRosales MarcialRosales force-pushed the issue-12324 branch 3 times, most recently from eab098f to 46aadeb Compare February 11, 2025 11:07
@MarcialRosales MarcialRosales added this to the 4.1.0 milestone Feb 11, 2025
@MarcialRosales
Support keycloak custom format via configuration
1179d3a
@MarcialRosales
Support in code the old keycloak format
3041d6c 
That was not keycloak format it was an
extension to the oauth spec introuduced
a few years ago. To get a token from
keycloak using this format, a.k.a.
requesting party token, one has to specify
a different claim type called
urn:ietf:params:oauth:grant-type:uma-ticket
@MarcialRosales
Clean up
ecacf0f
@MarcialRosales MarcialRosales marked this pull request as ready for review February 11, 2025 15:52
@MarcialRosales
Apply fix from PR #13180
bf7de92
@MarcialRosales
Fix test case name
aeda3ca 
Fix test case nam# Por favor ingresa el mensaje del commit para tus cambios. Las
@michaelklishin michaelklishin merged commit fdaf16b into main Feb 13, 2025
270 checks passed
@michaelklishin michaelklishin deleted the issue-12324 branch February 13, 2025 01:26
@mergify mergify bot mentioned this pull request Feb 13, 2025
Merged
6 tasks
michaelklishin added a commit that referenced this pull request Feb 13, 2025
@michaelklishin
Resolve a conflict #13216 #12324 #13180
e08aa4b
michaelklishin added a commit that referenced this pull request Feb 13, 2025
@michaelklishin
Merge pull request #13259 from rabbitmq/mergify/bp/v4.1.x/pr-13216
6064335 
Support keycloak custom format via configuration (backport #13216)
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 bazel
Projects
None yet
Milestone
4.1.0
Development

Successfully merging this pull request may close these issues.
2 participants
@MarcialRosales @michaelklishin