Skip to content

Resynchronisation des groups à la reconnexion d'un provider - #158

Open
MoulinZ wants to merge 2 commits into
PnX-SI:developfrom
RNF-SI:feature/keycloak-provider
Open

Resynchronisation des groups à la reconnexion d'un provider#158
MoulinZ wants to merge 2 commits into
PnX-SI:developfrom
RNF-SI:feature/keycloak-provider

Conversation

@MoulinZ

@MoulinZ MoulinZ commented Apr 9, 2026

Copy link
Copy Markdown

Jusqu'à présent, les groupes en provenance d'un provider externe étaient appairé à la première connexion, mais pas mis à jour aux connexions suivantes. C'est maintenant le cas.
J'ai fait des tests également :

  • Un pour checker qu'un groupe enlevé était bien enlevé à la nouvelle connexion (mirroir strict)
  • Le deuxième pour checker qu'un group qui n'est pas dans les appariements de group_mapping mais qui aurait été ajouté manuellement soit bien ignoré

Par contre j'ai pas réussi à lancer les tests sur le submodule...

@MoulinZ

MoulinZ commented Apr 9, 2026

Copy link
Copy Markdown
Author

Bon, je suis pas très doué, j'ai ajouté après le commit pour ajouter une route dédiée à la connexion sur mobile, je savais pas qu'elle viendrait directement dans la pull request... dites moi si c'est ok où s'il faut refaire quelque chose

@jacquesfize

Copy link
Copy Markdown
Contributor

Salut @MoulinZ, on va regarder ça :) Tu peux pointer sur develop ?

@MoulinZ

MoulinZ commented Apr 9, 2026

Copy link
Copy Markdown
Author

ah bah oui... faut refaire la pr ?

@jacquesfize
jacquesfize changed the base branch from master to develop April 9, 2026 15:40
@jacquesfize

Copy link
Copy Markdown
Contributor

ah c'est bon, j'ai réussi à le faire

@jacquesfize

Copy link
Copy Markdown
Contributor

Bon, je suis pas très doué, j'ai ajouté après le commit pour ajouter une route dédiée à la connexion sur mobile

Je viens de voir ce message x) Ca serait pas mal de séparer les deux.

@MoulinZ
MoulinZ force-pushed the feature/keycloak-provider branch 2 times, most recently from dd02b1b to e41dfa4 Compare May 4, 2026 11:57
@MoulinZ

MoulinZ commented May 4, 2026

Copy link
Copy Markdown
Author

Je crois que c'est ok, il n'y a plus le commit dd02b1b

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.

2 participants