Skip to content

Tech: Mise à jour de l'API SIRENE [GEN-2557]#6406

Merged
tonial merged 1 commit into
masterfrom
alaurent/sirene
Jun 26, 2025
Merged

Tech: Mise à jour de l'API SIRENE [GEN-2557]#6406
tonial merged 1 commit into
masterfrom
alaurent/sirene

Conversation

@tonial

@tonial tonial commented Jun 24, 2025

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Il y a une PR de secrets pour mettre à jours les settings (ajout uniquement, on supprimera les anciens settings après le déploiement)

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?
  • Ajouter l'étiquette « Bug » ?

🏝️ Comment tester ?

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

@tonial tonial self-assigned this Jun 24, 2025
@tonial tonial added the modifié Modifié dans le changelog. label Jun 24, 2025
@notion-workspace

Copy link
Copy Markdown

Comment thread config/settings/base.py
Comment thread config/settings/base.py
API_INSEE_CLIENT_ID = os.getenv("API_INSEE_CLIENT_ID")
API_INSEE_CLIENT_SECRET = os.getenv("API_INSEE_CLIENT_SECRET")
API_INSEE_USERNAME = os.getenv("API_INSEE_USERNAME")
API_INSEE_PASSWORD = os.getenv("API_INSEE_PASSWORD")

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Quand je vais sur Comment interroger l'API Sirene . Ça semble parler de se créer une clef d'API "On peut la renouveler ou la révoquer. Elle n’a pas de durée limite." et il suffirait ensuite de la passer en --header 'X-INSEE-Api-Key-Integration: xxxxxxx' ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

On utilise l'api privée, et non la publique

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Hum... si ça se trouve on peut se contenter de la publique, je vais essayer de comprendre la différence

@tonial tonial Jun 24, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Alors ça fonctionne, mais j'ai l'impression que le délai de réponse est catastrophique : si je redemande une siret déjà interrogé, ça mouline super longtemps.

Comment thread config/settings/base.py
@tonial tonial force-pushed the alaurent/sirene branch from 5b0a3bd to bd01765 Compare June 25, 2025 13:27
@tonial tonial added this pull request to the merge queue Jun 26, 2025
Merged via the queue into master with commit ab8c889 Jun 26, 2025
14 checks passed
@tonial tonial deleted the alaurent/sirene branch June 26, 2025 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants