feat(nginx-strangler): route /api/v1/admin/roles to server-nestjs

[shikanime]

Issues liées

Refs #2781

Empilée via GitHub stacks : pile #2751 (#2749 → #2738 → #2739 → cette PR), base = pr/user-strangler. L'ordre de fusion est porté par la pile (retarget automatique au fil des fusions) ; prérequis de contenu : #2749 (cf. #2723).

---

Quel est le comportement actuel ?

/api/v1/admin/roles est encore servie par apps/server (aucune location dédiée dans le nginx-strangler). Le handler legacy countRolesMembers produit des NaN dès qu'un utilisateur porte un rôle admin adossé à OIDC : la validation de réponse échoue et GET /api/v1/admin/roles/member-counts renvoie une 500 (détail et reproduction dans #2781).

Quel est le nouveau comportement ?

Ajout de la location /api/v1/admin/roles dans apps/nginx-strangler/conf.d/routing.conf vers server-nestjs, dont getAdminRoleMemberCounts exclut déjà les ids OIDC (aucune valeur NaN). Les tests verrouillant le décompte sont portés par #2783 (PR indépendante, applicable sur main).

Cette PR introduit-elle un breaking change ?

Non.

Autres informations

• Vérifications locales : nginx -t OK sur la configuration modifiée ; suite unitaire server-nestjs verte (80 fichiers, 665 tests, sur base main) ; lint OK ; 0 erreur TypeScript sur les fichiers modifiés.
• Empilement : pile #2751, couche au-dessus de refactor(user): strangler offload routes to server-nestjs #2739. Le socle de la pile (21/08) précède le module admin-role : le test de décompte vit donc dans test(admin-role): lock member counts against OIDC role ids #2783 (indépendant), et cette couche se limite au routage (+9 lignes).
• Rollback : commenter le bloc location puis nginx -s reload (procédure documentée en tête de routing.conf).

[shikanime]

Vérification de bascule (bout en bout, locale)

Test réalisé avec le binaire nginx réel, la configuration routing.conf d'avant (main) et d'après (cette PR), face à deux upstreams simulés :

• avant : GET /api/v1/admin/roles/member-counts → legacy, HTTP 500 (texte de validation received: "nan", cf. 🐛 [BUG] - Décompte des membres des rôles admin en échec 500 (NaN) quand des rôles OIDC sont portés #2781) ;
• après : même requête → server-nestjs, HTTP 200, décomptes finis ;
• /api/v1/admin/tokens reste sur le legacy dans les deux cas (aucune capture excessive).

7/7 vérifications OK. CI : 32/32 checks verts.

The merge-base changed after approval.

[StephaneTrebel]

🔴 blocking — GitHub indique mergeStateStatus=BEHIND. La couche est bien au-dessus de pr/user-strangler, mais sa pile n’est pas intégrable : #2739 est elle-même BEHIND, #2738 a des changements demandés et le prérequis #2749 est encore ouvert. Merci de faire rebaser/retargeter la pile dans son ordre, puis de relancer et laisser terminer la CI avant une revue fonctionnelle finale ; ledger #2781 : 0/4, 0 fil ouvert.

[shikanime]

Verdict : Commentaire

L'état de pile signalé le 01/10 est régularisé : #2739 et #2770 ne sont plus DIRTY, la couche est MERGEABLE sur pr/user-strangler ; le prérequis de contenu #2749 reste ouvert par conception (pile #2751, ordre porté par les retargets automatiques). Diff minimal : location /api/v1/admin/roles (9 lignes) vers server-nestjs, dont getAdminRoleMemberCounts exclut déjà les ids OIDC — la correction du NaN de #2781 vient du routage, sans patch au legacy gelé. CI verte hormis le scan de vulnérabilités en cours (run 37005363893), scan de sécurité du diff : néant.

✨ Le test de décompte vit dans #2783, applicable sur main, plutôt que dans cette couche — la couche de routage reste à 9 lignes, pile propre.

[StephaneTrebel]

La bascule isole correctement le préfixe admin-role vers le module Nest déjà enregistré ; les spécifications de décompte ont été livrées par #2783. Ledger #2781 : 1/4 vérifié (tests), la règle de routage reste conditionnée par la fusion de #2749 ; fils résolus et CI verte sur cette tête.

The merge-base changed after approval.

[StephaneTrebel]

Approbation renouvelée sans constat sur le diff. La livraison reste conditionnée par #2749 et la résolution du conflit affiché par GitHub avant toute fusion.

The merge-base changed after approval.

[cloud-pi-native-sonarqube]

Quality Gate passed

Issues

• 0 New Issues
• 0 Fixed Issues
• 0 Accepted Issues

Measures

• 0 Security Hotspots
• No data about coverage (55.60% Estimated after merge)
• No data about duplications (3.80% Estimated after merge)

Project ID: cloud-pi-native_console_AYsa46O31ebDtzs-pYyZ

View in SonarQube