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] Création de l’API MADDO (PIX-16420) #11354

Merged
merged 10 commits into from
Feb 17, 2025

Conversation

nlepage
Copy link
Member

@nlepage nlepage commented Feb 6, 2025

🍔 Problème

Les données mises à disposition à l'extérieur de Pix le sont via l'API utilisée par le front.
Ceci crée une disparité d'utilisation de l'API, entre des usages de données live par le front, rapides, et des usages de données froides, potentiellement lents, mal gérée par l'hébergement de nos apps.
Ce qui a pour conséquence :

  • une baisse de performance sur certaines requêtes, des files d'attente longues provoquant des erreurs 504.
  • un pool de connexion inutilement important pour la base de données froides (min. 1 connexion par conteneur) et donc un plan trop élevé pour l'usage réel.

🍟 Proposition

Isoler les usages dans 2 applications différentes :

  • l'API Pix continue à servir les routes utilisées par les fronts
  • l'API MaDDo sert uniquement les routes accessibles via la gateway.

image

🧃 Remarques

N/A

😋 Pour tester

https://pix-api-maddo-review-pr11354.osc-fr1.scalingo.io/api/

@pix-bot-github
Copy link

pix-bot-github commented Feb 7, 2025

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

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

api/server.maddo.js Outdated Show resolved Hide resolved
Co-authored-by: Ce que tu veux <[email protected]>
Co-authored-by: Vincent Hardouin <[email protected]>
HEYGUL and others added 9 commits February 17, 2025 10:54
Co-authored-by: Nicolas Lepage <[email protected]>
Co-authored-by: Vincent Hardouin <[email protected]>
Co-authored-by: Guillaume Lagorce <[email protected]>
Co-authored-by: Nicolas Lepage <[email protected]>
Co-authored-by: Guillaume Lagorce <[email protected]>
Co-authored-by: Nicolas Lepage <[email protected]>
Co-authored-by: Guillaume Lagorce <[email protected]>
Co-authored-by: Nicolas Lepage <[email protected]>
Co-authored-by: Ce que tu veux <[email protected]>
Co-authored-by: Vincent Hardouin <[email protected]>
Co-authored-by: Nicolas Lepage <[email protected]>
Co-authored-by: Vincent Hardouin <[email protected]>
Co-authored-by: Aurélie Crouillebois <[email protected]>
Co-authored-by: Ce que tu veux <[email protected]>
Co-authored-by: Vincent Hardouin <[email protected]>
@pix-service-auto-merge pix-service-auto-merge merged commit 76f66ad into dev Feb 17, 2025
9 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-16420-maddo-server branch February 17, 2025 10:01
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