Repository navigation
refactor(user): strangler offload routes to server-nestjs - #2739
Conversation
785dab3 to
82be36e
Compare
82be36e to
dfd3ef2
Compare
|
🤖 Hey ! A preview of the application is available at : https://console-pr-2739.dso.cpin-hp.numerique-interieur.fr Please be patient, deployment may take a few minutes. |
![github-code-quality[bot]](https://avatars.githubusercontent.com/u/9919?s=60&v=4){kind=small}
52d0c1a to
1ca3880
Compare
|
PR réduite à la bascule nginx. Le doublon Diff actuel : |
1ca3880 to
d2adcae
Compare
4d404ea to
b1942a2
Compare
b1942a2 to
e3b8041
Compare
shikanime
left a comment
shikanime
left a comment
There was a problem hiding this comment.
Verdict : Commentaire
Le DIRTY signalé le 01/10 est levé : la branche est MERGEABLE sur pr/user-migration (BEHIND uniquement — le retarget de pile est porté par la stack #2751). Le diff tient en une location /api/v1/users de 9 lignes vers server-nestjs, exactement le gabarit des bascules précédentes, rollback documenté en tête de routing.conf. CI verte à la tête (run 37005360077), scan de sécurité : néant.
✨ Extraire la bascule nginx de la migration du module (#2738) permet de rejouer le cutover indépendamment du code NestJS — découpage de risque correct.
e3b8041 to
55a2fcd
Compare
StephaneTrebel
left a comment
StephaneTrebel
left a comment
There was a problem hiding this comment.
La cible NestJS conserve désormais l’accès public à /api/v1/users/matching ; la bascule nginx ne réintroduit plus la régression signalée. Fils résolus et CI verte.
The merge-base changed after approval.
7397a67 to
cdc8cca
Compare
020a40b to
93bbf69
Compare
93bbf69 to
54d26b5
Compare
Co-authored-by: Automata <automata@shikanime.studio>
54d26b5 to
aa94bf6
Compare
|
0 New Issues
0 Fixed Issues
0 Accepted Issues
No data about coverage (55.60% Estimated after merge)
StephaneTrebel
left a comment
StephaneTrebel
left a comment
There was a problem hiding this comment.
Approbation renouvelée : la migration user désormais intégrée conserve l’accès public de /api/v1/users/matching ; la bascule nginx ne réintroduit pas la régression précédemment signalée.
2 participants
Issues liées
Refs #1889
Quel est le comportement actuel ?
/api/v1/usersest servie par l'ancienne application Fastifyapps/servervia le catch-all/api/de nginx-strangler.Quel est le nouveau comportement ?
La route est basculée vers
server-nestjsdansapps/nginx-strangler/conf.d/routing.conf(location /api/v1/users). Empilée sur la migration du moduleuser(#2738) : la bascule ne devient effective qu'une fois ce module fusionné.Cette PR introduit-elle un breaking change ?
Non.
Autres informations
Recréée pour #1889 (remplace #2499, non réouvrable après suppression de branche). Rollback selon le README : commenter le bloc (
# ROLLBACK AAAA-MM-JJ) puisnginx -s reload.