Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

Remove the routes only the legacy site used, check health at /health, and stop serving Canada, Nigeria and Israel

Ouverte
  • Quelque chose ne va pas ?
#4,016 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 3 jours

@anth-volk y travaille déjà.

Depuis le 22/10/2026.

  • #4017 par @anth-volk — ouverte

Évaluation

Difficulté
4/5
Temps estimé
3-5 jours
Accessibilité débutants
25/100
Type d'issue
Refactorisation
Clarté
Clairement spécifiée
Activité
À l'abandon
Stack technique
fastapi, google-cloud, python
Domaine
api, backend, infrastructure

Piste de recherche

Start by reading the route definitions, health check and startup probe configuration, deployment health checks, country list and model package dependencies. Check the tests and OpenAPI document that cover the listed routes and countries. The issue is complete when the legacy routes and health endpoints are removed, /health serves the probes, only US and UK remain, and the pre-merge traffic checks pass; linked pull request #4017 is already open.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

What

Remove what only legacy.policyengine.org or the API documentation used, move Cloud Run health checks to /health, and stop serving Canada, Nigeria and Israel.

  • Routes removed: /{country_id}/user-policy (POST, GET, PUT), /{country_id}/user-profile (POST, GET, PUT), GET /{country_id}/policies, GET /simulations, GET / and GET /specification. POST /{country_id}/calculate stays: it still serves about 140,000 requests a month, and its callers move first in a later change.
  • Health checks: /health reports ready only once the startup warmup has finished and the runtime configuration is valid (503 before), taking over from /readiness-check. The Cloud Run startup probe and the deployment's health checks poll it, and /liveness-check and /readiness-check are removed.
  • Countries: the v1 routes serve only the US and the UK, their country lists derive from the one country list in code, and the Canada, Nigeria and Israel model packages are no longer installed.

Why

The legacy site is being retired, and production request logs for the 30 days to 2026-10-19 show little else calling these routes: no requests for saved policies or user profiles, 13 for the policy search, 11 for the recent-simulations list, and about 24,300 for GET /, from an uptime monitor and link previews. Keeping them means keeping code, tests and a hand-written OpenAPI document that nothing uses. Two health-check routes beside /health make it unclear which one deployments rely on. About 1,160 requests named the three countries in those 30 days, 784 of them from the legacy site and 49 from app.policyengine.org (which stops offering them first); dropping their packages lowers the API's memory use.

Before merging

  • legacy.policyengine.org is no longer served, and request logs show no remaining traffic to the removed routes or the three countries.
  • The external uptime monitor polling GET / is moved to /health.
Langage dominant
Python
Étoiles
18
Forks
33
Merge moyen
2 j 18 h
PR mergées (30 j)
36

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de PolicyEngine/policyengine-api

Toutes les issues de PolicyEngine/policyengine-api

Issues similaires

Plus d'issues Python

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.