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

[FEATURE] Ajouter un feature toggle pour le confinement des access tokens (PIX-15924) #11063

Merged
merged 1 commit into from
Jan 13, 2025

Conversation

theotime2005
Copy link
Contributor

@theotime2005 theotime2005 commented Jan 10, 2025

🎄 Problème

On souhaite avoir un feature toggle permettant de tester une future implémentation d'access tokens.

🎁 Proposition

Ajouter le feature toggle et le définir à false par défaut sur la RA.

🧦 Remarques

La variable isUserTokenAudConfimentEnabled a été placée dans config.featureToggles même si elle n'est pas utilisée dans le front. On a choisi de ne pas créer un objet spécialement pour cette variable.

🎅 Pour tester

Avec la valeur sur False

Faire cette requête dans un terminal:

curl -X GET --location "https://api-pr11063.review.pix.fr/api/feature-toggles"

Vérifier que cette ligne est présente:

"is-user-token-aud-confinement-enabled": false,

Avec la valeur sur true

  • Se rendre sur le scalingo des variables d'env et passer FT_USER_TOKEN_AUD_CONFINEMENT_ENABLED à true
  • Relancer le déploiement
  • Faire cette requête dans un terminal:
curl -X GET --location "https://api-pr11063.review.pix.fr/api/feature-toggles"

Vérifier que cette ligne est présente:

"is-user-token-aud-confinement-enabled": true,

@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 :

@theotime2005 theotime2005 self-assigned this Jan 10, 2025
@theotime2005 theotime2005 force-pushed the pix-15924-add-a-new-feature-toggle branch 2 times, most recently from 747e8bc to 9469b7e Compare January 10, 2025 12:33
Copy link
Contributor

@bpetetot bpetetot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revue et test OK

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-15924-add-a-new-feature-toggle branch from a7a37e1 to c0e7a14 Compare January 13, 2025 08:52
@pix-service-auto-merge pix-service-auto-merge merged commit 40bf296 into dev Jan 13, 2025
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-15924-add-a-new-feature-toggle branch January 13, 2025 08:58
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.

6 participants