-
Notifications
You must be signed in to change notification settings - Fork 311
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
Ordering executions with keycloak_authentication_execution #890
Comments
Hi. You are right, the Keycloak's Admin API documentation does imply that you can provide a full executor representation. The method responsible for the priority looks like this: private int getNextPriority(AuthenticationFlowModel parentFlow) {
List<AuthenticationExecutionModel> executions = realm.getAuthenticationExecutionsStream(parentFlow.getId())
.collect(Collectors.toList());
return executions.isEmpty() ? 0 : executions.get(executions.size() - 1).getPriority() + 1;
} And the API endpoint handlers always use that. This can be seen in the Keycloak's source code:
Meaning that it's impossible to set explicit priority value for an executor in a flow with current Keycloak Admin API implementation. |
Hi, Thanks for spending time on this and explaning it very well. Actually, I realised how priority has no impact on the API end after creating this whilst having a deep dive on Keycloak's API code. What they only offer is increase/decrease priority endpoints. Which is almost impossible to implement (at least in a clean and tidy way instead of creating too many hacky solutions). Thus, I commented on an another issue (linked to this one) on the Keycloak's repo and explained it to them. However, a bot commented 4 days ago with saying there are too many issues at the moment and we cannot prioritise this currently. I didn't want to close this one for not to loose the track of it. You could be aware the issue is solved on the other end and this could progress. Cheers, |
I believe this can be closed as a duplicate of #296 |
Hi there,
In the documentation it stresses how the execution order works as it follows
If the realm or the resource is created for the first time, it works perfectly. However, those orders doesn't have Save button after making a change and change is applied immediately if a person drags&drops any of the items in the list accidentally. Also, the next terraform apply wouldn't fix that problem unless the execution flow is deleted and terraform applied again.
I'd like to point out I can see there is "
priority
" object in the realm exports.Also, Keycloak admin API documentation suggests that it accepts
priority
in here.I believe this can be implemented in the provider end.
Thanks,
Levent.
The text was updated successfully, but these errors were encountered: