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

Adaptation à la nouvelle API INSEE #1468

Merged
merged 3 commits into from
Jan 16, 2025
Merged

Adaptation à la nouvelle API INSEE #1468

merged 3 commits into from
Jan 16, 2025

Conversation

alemangui
Copy link
Collaborator

@alemangui alemangui commented Jan 14, 2025

Closes #1433

Contexte

L'ancien portal API INSEE fermera le 28 fév 2025. Cette PR s'adapte à la nouvelle API.

Scope

  • Il n'est pas nécessaire de calculer le token avant de faire la requête, j'ai donc supprimé le modèle InseeToken
  • L'auth se fait via l'entête X-INSEE-Api-Key-Integration - désormais utilisé dans external_utils

Compte INSEE

J'ai du créer un compte sur https://portail-api.insee.fr/ en utilisant l'email de l'équipe compl-alim.

J'ai créé trois applications différentes pour chacun de nos environnements : prod, staging et démo.

Démo

Screencast.from.2025-01-14.08-50-56.mp4

Copy link
Collaborator

@pletelli pletelli left a comment

Choose a reason for hiding this comment

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

Ok pour moi !
Merci d'avoir suivi ça.
On a donc 3 INSEE_API_KEY un pour chaque env. Est-ce que tu veux le mettre en commentaire dans config/settings.py ?

@alemangui
Copy link
Collaborator Author

@pletelli bonne idée. Je l'ai ajouté dans le README dans la section des variables d'environnement (commit 49a522e)

@alemangui alemangui merged commit 5a7a475 into staging Jan 16, 2025
5 checks passed
@alemangui alemangui deleted the 1433-api-insee branch January 16, 2025 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migration API INSEE
2 participants