Repository navigation
feat(nginx-strangler): basculer admin tokens vers server-nestjs - #2930
Conversation
StephaneTrebel
left a comment
StephaneTrebel
left a comment
There was a problem hiding this comment.
No findings. Le diff est limité au routage des tokens administrateur : nginx -t et les trois sondes HTTP GET/POST/DELETE vers des upstreams témoins passent ; contrôleur chargé, permissions ListAdminToken/ManageAdminToken et sérialisation du contrat vérifiés, avec les tests ciblés verts. Le suivi pertinent est #2922 (pas de lien Development actuellement) : aucune case distante modifiée, aucune recette E2E sur intégrations réelles revendiquée et CI encore en cours au relevé.
a192ec9 to
7b4d27b
Compare
StephaneTrebel
left a comment
StephaneTrebel
left a comment
There was a problem hiding this comment.
No findings. Depuis la tête approuvée, le seul changement est une ligne vide de séparation dans routing.conf ; les arbres applicatifs sont identiques et les preuves de parité précédentes restent applicables (66 tests ciblés déjà vérifiés, non réexécutés pour ce changement de présentation). nginx -t et les 3 sondes HTTP de routage vers des upstreams témoins viennent de repasser, CI verte et aucun fil ouvert ; suivi fonctionnel #2922 conservé sans modification du ledger ni nouvelle recette E2E des intégrations externes.
7b4d27b to
f6d8119
Compare
StephaneTrebel
left a comment
StephaneTrebel
left a comment
There was a problem hiding this comment.
No findings. Le bloc admin-token est inchangé depuis la précédente approbation et le backend/chargement du module est identique ; nginx -t et les trois sondes HTTP GET/POST/DELETE avec upstreams témoins viennent de repasser. Les preuves de contrats/permissions déjà établies restent applicables, sans réexécution des tests applicatifs ni nouvelle recette E2E ; suivi #2922 conservé sans mutation du ledger, aucun fil ouvert et CI verte.
shikanime
changed the title
shikanime
changed the title
shikanime
changed the title
f6d8119 to
be04c8d
Compare
yorha-operator
left a comment
yorha-operator
left a comment
There was a problem hiding this comment.
Approbation sur le head post-conflit (3314d49).
Ajouter le bloc location /api/v1/admin/tokens routant vers server-nestjs dans apps/nginx-strangler/conf.d/routing.conf. Co-authored-by: Automata <automata@shikanime.studio> Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr> Change-Id: I2c911e36db121f5376667af4776cf9246a6a6964
ea35a96 to
637675a
Compare
Le formulaire client emet une chaine vide lorsque la date d expiration est absente ; le schema Zod rejete desormais cette valeur en 400 la ou le legacy l acceptait, ce qui fait echouer le test Playwright de creation de token admin en merge queue. Co-authored-by: Automata <automata@shikanime.studio> Signed-off-by: William Phetsinorath <william.phetsinorath-open@interieur.gouv.fr>
c56bd71 to
c8db3bc
Compare
|
0 New Issues
0 Fixed Issues
0 Accepted Issues
Pourquoi
L'audit du 23 octobre 2026 de #1889 confirme que les 3 routes
/api/v1/admin/tokensatteignent encoreapps/servervia nginx-strangler. Elles doivent être déléguées àserver-nestjs.Ce qui a été fait
Ajout d'un bloc
location /api/v1/admin/tokensdansapps/nginx-strangler/conf.d/routing.confqui route versserver-nestjs.Les méthodes
GET,POSTetDELETEde/api/v1/admin/tokenssont servies par NestJS via nginx-strangler, sans passage par le legacy.Vérification
apps/nginx-strangler/conf.d/routing.conf, +10 lignes (bloclocationajouté + 1 ligne de séparation).apps/nginx-strangler/conf.d/routing.conf(indentation,proxy_pass, headersHost,X-Real-IP,X-Forwarded-For,X-Forwarded-Proto).server-nestjs(admin-token/admin-token.controller.ts) et le moduleAdminTokenModuleest importé dansMainModule.nginx -tn'a pas pu être exécuté dans cet environnement (nginx non installé) ; la validation doit être faite localement avant merge.Références
serverversserver-nestjs#1889 — audit et suivi de la bascule complète