Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@mposolda
Copy link
Contributor

@mposolda mposolda commented Jul 12, 2024

…cloak 25

closes #31224

Some additional notes to this PR:

  • Before Keycloak 15, the offline tokens had only the claim session_state, but not sid
  • In Keycloak 15, there was duplicated claim sid added in addition to session_state. Related commit: 8c1ea60b
  • In Keycloak 25, the claim session_state was removed and only sid was kept. Related commit: b4f791b6

So migration of offline tokens is broken from Keycloak versions 14 or earlier to Keycloak 25 versions. This means that fix requires backport to Keycloak 25.0.X as well.

This issue is not caught by the MigrationTest, which tests migration of offline tokens, but just from some recent Keycloak version. In this PR, I've added OfflineTokenMigrationTest to simulate migration of old offline tokens to the latest Keycloak.

Copy link
Contributor

@graziang graziang left a comment

Choose a reason for hiding this comment

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

@mposolda Thanks for the PR, LGTM!

Copy link
Contributor

@ahus1 ahus1 left a comment

Choose a reason for hiding this comment

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

Approving based on @jonkoops' and @graziang's reviews.

@mposolda
Copy link
Contributor Author

@graziang @jonkoops @ahus1 Thanks everyone!

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.

Offline tokens created in Keycloak 9 will not work on Keycloak 25

4 participants