Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TECH] Rendre le code lié à l'authentification des applications plus clair (PIX-15876) #10906

Merged
merged 4 commits into from
Dec 30, 2024

Conversation

lego-technix
Copy link
Contributor

@lego-technix lego-technix commented Dec 27, 2024

🎄 Problème

Le code lié à l'authentification des applications n'est pas très clair à lire, notamment pour les valeurs qui se trouvent dans le fichier api/src/shared/config.js où on distingue mal, notamment dans les diff des PR ou de l'historique Git, quelles sont les valeurs utilisées en production des valeurs surchargées pour les tests.

🎁 Proposition

  • Pour la partie de la configuration dédiée aux tests (process.env.NODE_ENV === 'test'), rendre les valeurs surchargées pour les tests explicites en les faisant commencer par le préfixe test-
  • Pour la partie de la configuration dédiée aux tests (process.env.NODE_ENV === 'test'), surcharger uniquement les propriétés qui ont besoin de l’être
  • Déplacer les fonctions privées en fin de fichier

🧦 Remarques

RAS

🎅 Pour tester

  • Vérifier que la CI passe
  • Vérifier que les tests en local passent

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@lego-technix lego-technix requested a review from yaf December 27, 2024 15:11
@lego-technix lego-technix changed the title [TECH] Rendre le code lié à l'authentification des applications plus clair [TECH] Rendre le code lié à l'authentification des applications plus clair (PIX-15876) Dec 30, 2024
@pix-service-auto-merge pix-service-auto-merge force-pushed the make-authenticate-application-clearer branch from 692e9c8 to 90a4c2d Compare December 30, 2024 16:20
@pix-service-auto-merge pix-service-auto-merge merged commit 49d32f8 into dev Dec 30, 2024
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the make-authenticate-application-clearer branch December 30, 2024 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants