Skip to content

[TECH] Suppression du token d’accès aux résultats de campagne (PIX-16610) #11443

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

Merged
merged 1 commit into from
Feb 18, 2025

Conversation

nlepage
Copy link
Member

@nlepage nlepage commented Feb 18, 2025

🥞 Problème

À chaque fois qu’une campagne est récupérée dans PixOrga, un token d’accès aux résultats de campagne est généré, mais ce token n’est plus utilisé par PixOrga.

🥓 Proposition

Supprimer le token.

🧃 Remarques

La route de récupération des résultats de campagne est GET /api/campaigns/{campaignId}/csv-assessment-results.

La variable d’environnement CAMPAIGN_RESULT_ACCESS_TOKEN_LIFESPAN pourra être supprimée sur les différents environnements.

😋 Pour tester

Vérifier qu’on peut toujours télécharger les résultats CSV d’une campagne depuis PixOrga.

@nlepage nlepage self-assigned this Feb 18, 2025
@nlepage nlepage requested a review from a team as a code owner February 18, 2025 11:09
@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 :

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.

Ok au niveau du code côté Accès

@machestla
Copy link
Contributor

Func & Tech review ok ✅

@nlepage nlepage added Tech Review OK 🚀 Ready to Merge Func Review OK PO validated functionally the PR and removed 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally labels Feb 18, 2025
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-16610-remove-campaign-results-token branch from 3a55ad7 to 9aed4ff Compare February 18, 2025 14:31
@pix-service-auto-merge pix-service-auto-merge merged commit b526f48 into dev Feb 18, 2025
10 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-16610-remove-campaign-results-token branch February 18, 2025 14:40
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.

8 participants