-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Closed
Labels
area/corekind/bugCategorizes a PR related to a bugCategorizes a PR related to a bugpriority/importantMust be worked on very soonMust be worked on very soonrelease/25.0.2release/26.0.0team/core-shared
Milestone
Description
Before reporting an issue
- I have read and understood the above terms for submitting issues, and I understand that my issue may be closed without action if I do not follow them.
Area
core
Describe the bug
When the return type of getExp() in org.keycloak.models.SingleUseObjectKeyModel was changed from an int to a Long, the old int behaviour was kept as a getter getExpiration() which has a default implementation to get the int value from getExp().
This causes the JSON serialization to include the value from getExpiration() as a claim expiration. Since the DefaultActionToken extends DefaultActionTokenKey which implements SingleUseObjectKeyModel, a `expiration claim is now included in all action tokens.
Version
25.0.1
Regression
- The issue is a regression
Expected behavior
Action tokens should have a claim exp for expiration.
Actual behavior
Action tokens has both exp and expiration claims for expiration.
How to Reproduce?
Use the forgotten password method and decode the reset password token. It has both exp and expiration.
Anything else?
No response
Metadata
Metadata
Labels
area/corekind/bugCategorizes a PR related to a bugCategorizes a PR related to a bugpriority/importantMust be worked on very soonMust be worked on very soonrelease/25.0.2release/26.0.0team/core-shared