Skip to content

test(server-nestjs): parité reconcile-only des rôles admin keycloak + garde d'émission - #2581

Draft
shikanime wants to merge 3 commits into
mainfrom
test/keycloak-admin-role-parity
Draft

test(server-nestjs): parité reconcile-only des rôles admin keycloak + garde d'émission#2581
shikanime wants to merge 3 commits into
mainfrom
test/keycloak-admin-role-parity

Conversation

@shikanime

Copy link
Copy Markdown
Member

Issues liées

#2574 #2576


Quel est le comportement actuel ?

Legacy déclenche hook.adminRole.upsert/delete à chaque create/patch/delete de rôle admin ; server-nestjs ne synchronise les groupes OIDC que via le cron reconcile, sans chemin évènementiel ni module admin-role.

Quel est le nouveau comportement ?

  • keycloak.service.spec.ts : +35 lignes — describe('migration parity: admin-role event path') : (a) verrouille la sync des groupes OIDC via cron uniquement, (b) garde négative « pas d'emitAdminRoleEvent » sur AppEventsService pour forcer tout futur chemin évènementiel à passer par l'audit de parité.

12/12 tests verts sur Node 24.

Cette PR introduit-elle un breaking change ?

Non.

Autres informations

Blocant cutover latent tracé côté migration (matrice u5/u9).

shikanime and others added 2 commits August 25, 2026 22:36
…e parity

Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr>
Change-Id: Iecda25d80f34897d6c421615f94040536a6a6964
… emit guard

Co-authored-by: Automata <automata@shikanime.studio>
Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr>
Change-Id: I44ee5ab058dc2603ab3b15d13f9daabc6a6a6964
@cloud-pi-native-sonarqube

Copy link
Copy Markdown

Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr>
Change-Id: I3a56c611ef662d54bfa0318d57c9c09b6a6a6964
@shikanime shikanime closed this Aug 28, 2026
@shikanime
shikanime deleted the test/keycloak-admin-role-parity branch August 28, 2026 15:49
@shikanime shikanime reopened this Aug 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant