Add EIDAS level and FranceConnect environment options in admin console #103
Conversation
fix InseeFr#74 Signed-off-by: Laurent Meunier <lme@atolcd.com>
|
|
||
| return ProviderConfigurationBuilder.create() | ||
| .property().name(ACEnvironment.ENVIRONMENT_PROPERTY_NAME) | ||
| .label("Environnement FranceConnect") |
There was a problem hiding this comment.
Idéalement, il faudrait pouvoir reprendre les libellés présents dans les fichiers src/main/resources/theme-resources/messages/admin-messages_*.properties, sauf que je ne vois pas comment faire.
Si qqun à une idée...
There was a problem hiding this comment.
Ca a l'air plus compliqué effectivement, on fera avec pour cette pr !
src/main/java/fr/insee/keycloak/providers/agentconnect/AgentConnectIdentityProviderConfig.java
Outdated
Show resolved
Hide resolved
src/main/java/fr/insee/keycloak/providers/agentconnect/AgentConnectIdentityProviderFactory.java
Outdated
Show resolved
Hide resolved
...ain/java/fr/insee/keycloak/providers/franceconnect/FranceConnectIdentityProviderFactory.java
Outdated
Show resolved
Hide resolved
|
PR testé avec succès dans un environnement local en Keycloak 24. |
|
Merci pour les tests. :) |
|
Super ! Content qu'on puisse enfin overrider l'interface d'admin de Keycloak ! On peut surement clean les anciens partials qui étaient utilisés par les anciennes versions de Keycloak pour modifier l'interface d'admin ( |
Signed-off-by: Laurent Meunier <lme@atolcd.com>
Signed-off-by: Laurent Meunier <lme@atolcd.com>
lme-atolcd
force-pushed
the
local-admin-conf
branch
from
ede0665
to
884870d
Compare
oui j'ai quasi la pr de nettoyage prete en testant, je l'envoie apres merge de cette pr. Je ferai la release ensuite. |
fix #74
Les URL utilisées correspondent bien à l'environnement FC sélectionné et le niveau eIDAS est bien pris en compte dans le paramètre
acr_values, ça semble fonctionner comme attendu, mais je n'ai pas pu réellement tester la modification (voir #102).