Skip to content

Conversation

ThomasKasene
Copy link
Contributor

@ThomasKasene ThomasKasene commented Aug 18, 2020

Fixes #22947

@pivotal-issuemaster
Copy link

@ThomasKasene Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-issuemaster
Copy link

@ThomasKasene Thank you for signing the Contributor License Agreement!

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Aug 18, 2020
@philwebb philwebb added type: task A general task and removed status: waiting-for-triage An issue we've not yet triaged labels Aug 18, 2020
@philwebb philwebb added this to the 2.4.x milestone Aug 18, 2020
@wilkinsona wilkinsona changed the title Added dependency management for nimbus-jose-jwt Dependency convergence error for nimbus-jose-jwt when depending on spring-boot-starter-oauth2-client Aug 25, 2020
@wilkinsona wilkinsona added type: bug A general bug and removed type: task A general task labels Aug 25, 2020
@wilkinsona wilkinsona modified the milestones: 2.4.x, 2.3.x Aug 25, 2020
wilkinsona pushed a commit that referenced this pull request Aug 25, 2020
@wilkinsona
Copy link
Member

@ThomasKasene Thanks very much for making your first contribution to Spring Boot.

@wilkinsona wilkinsona modified the milestones: 2.3.x, 2.3.4 Aug 25, 2020
@dreis2211
Copy link
Contributor

This breaks quite some tests unfortunately @wilkinsona . Could we revert this for the time being again?

@wilkinsona
Copy link
Member

wilkinsona commented Aug 25, 2020

So it does. Sorry, @dreis2211. I probably shouldn't be pushing changes in my sleep-deprived state…

@wilkinsona wilkinsona reopened this Aug 25, 2020
@dreis2211
Copy link
Contributor

We've all been there - no need to be sorry

@wilkinsona
Copy link
Member

Spring Security 5.3.4 uses nimbus-jose-jwt 8.19 so let's align with that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants