Skip to content

Conversation

@marcusdacoregio
Copy link
Contributor

@marcusdacoregio marcusdacoregio commented May 6, 2021

Update NimbusOpaqueTokenIntrospector and NimbusReactiveOpaqueTokenIntrospector to use MediaType.APPLICATION_JSON instead of the deprecated MediaType.APPLICATION_JSON_UTF8

Closes gh-9353

@marcusdacoregio marcusdacoregio requested review from jzheaux and rwinch May 6, 2021 13:23
@marcusdacoregio marcusdacoregio self-assigned this May 6, 2021
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label May 6, 2021
@jzheaux jzheaux added in: oauth2 An issue in OAuth2 modules (oauth2-core, oauth2-client, oauth2-resource-server, oauth2-jose) status: duplicate A duplicate of another issue type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels May 6, 2021
@jzheaux jzheaux added this to the 5.5.0 milestone May 6, 2021
Copy link
Contributor

@jzheaux jzheaux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR, @marcusdacoregio! I've left some feedback inline.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should there be a similar test on the servlet side?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The tests for NimbusOpaqueTokenIntrospector already use the MediaType.APPLICATION_JSON instead of the deprecated one so I guess there is no need to change them.

Update NimbusOpaqueTokenIntrospector and NimbusReactiveOpaqueTokenIntrospector to use MediaType.APPLICATION_JSON instead of the deprecated MediaType.APPLICATION_JSON_UTF8

Closes spring-projectsgh-9353
@marcusdacoregio marcusdacoregio changed the title Changes deprecated property in NimbusOpaqueTokenIntrospector Update Deprecated Property in Opaque Token Introspectors May 6, 2021
@rwinch rwinch removed their request for review May 6, 2021 19:06
@jzheaux jzheaux merged commit 6413511 into spring-projects:main May 6, 2021
@jzheaux
Copy link
Contributor

jzheaux commented May 6, 2021

Thanks, @marcusdacoregio! This is now merged into main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in: oauth2 An issue in OAuth2 modules (oauth2-core, oauth2-client, oauth2-resource-server, oauth2-jose) status: duplicate A duplicate of another issue type: bug A general bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

NimbusOpaqueTokenIntrospector uses deprecated MediaType.APPLICATION_JSON_UTF8

3 participants