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.

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

Refactor authorization header in CustomClaimConverter #24413

Merged
merged 1 commit into from
Nov 30, 2023
Merged

Refactor authorization header in CustomClaimConverter #24413

merged 1 commit into from
Nov 30, 2023

Conversation

yhao3
Copy link
Contributor

@yhao3 yhao3 commented Nov 30, 2023

  • Replaced the manual construction of the Authorization header with the use of setBearerAuth method.
  • Removed the buildBearer method as it is no longer needed.

Please make sure the below checklist is followed for Pull Requests.

When you are still working on the PR, consider converting it to Draft (below reviewers) and adding skip-ci label, you can still see CI build result at your branch.

@mraible
Copy link
Contributor

mraible commented Nov 30, 2023

@yhao3 Thanks for your contribution!

@mraible mraible merged commit 36d5d4d into jhipster:main Nov 30, 2023
48 checks passed
@deepu105 deepu105 added this to the 8.1.0 milestone Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants