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

Add priority field support in authentication flow executions #970

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gim-
Copy link

@gim- gim- commented Jun 1, 2024

Allows to declaratively define authentication flow execution and subflow priority in Keycloak 25 onwards.

Additionally I've removed raisePriority/lowerPriority functions as they don't seem to be used anywhere.

Fixes #296

@gim- gim- force-pushed the auth-flow-executor-priority branch 2 times, most recently from 9a4be06 to 9d4f775 Compare June 1, 2024 11:27
Signed-off-by: Andrejs Mivreniks <andrejs@fastmail.com>
@gim- gim- force-pushed the auth-flow-executor-priority branch from 9d4f775 to 0019a54 Compare June 1, 2024 12:07
@gim-
Copy link
Author

gim- commented Jun 1, 2024

Test run in GitHub actions appears to be flaky.

@gim- gim- marked this pull request as ready for review June 20, 2024 05:00
@berni2288
Copy link

Hey, is there anything that blocks this PR from being merged?
If I understood it right, the Github action test is just buggy.

@gim-
Copy link
Author

gim- commented Jul 8, 2024

@berni2288 The project is not maintained unfortunately - #964

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

Successfully merging this pull request may close these issues.

Allow for defining Flow priorities declaratively
2 participants