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

Upgrade to Openiddict 6.0.0 #20979

Merged
merged 23 commits into from
Dec 20, 2024
Merged

Upgrade to Openiddict 6.0.0 #20979

merged 23 commits into from
Dec 20, 2024

Conversation

@maliming maliming modified the milestones: 9.0-final, 9.1-preview Nov 4, 2024
@maliming maliming marked this pull request as ready for review December 9, 2024 05:19
@maliming maliming requested a review from EngincanV December 18, 2024 02:42
@kevinchalet
Copy link

Hey @maliming,

Hope you're doing well 😃

Something to keep in mind: OpenIddict automatically falls back to the old/legacy ept:logout and ept:device permissions if ept:end_session and ept:device_authorization permissions are not present when validating incoming requests (for backward compat' with older versions), but if you want your "application edit" views to behave similarly and correctly indicate that an app is allowed to use these two endpoints via the checkboxes, you'll probably need custom logic. Or a migration script that replaces ept:logout and ept:device by ept:end_session and ept:device_authorization in the DB.

Let me know if you have any question.

Happy holidays! 🎄

@maliming
Copy link
Member Author

Thanks @kevinchalet
Merry Christmas in advance! 🎄

@EngincanV EngincanV merged commit 66aef97 into dev Dec 20, 2024
3 checks passed
@EngincanV EngincanV deleted the openiddict6 branch December 20, 2024 08:19
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.

Upgrade to Openiddict 6.0.0
3 participants