Remove or guard /_test/* stub routes in production

Aperta
#18 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
58/100
Tipo di issue
Refactoring
Chiarezza
Abbastanza chiara
Stato di attività
Tranquilla
Stack tecnologico
typescript
Ambito
api, backend

Direzione di ricerca

Inizia dalle tre definizioni delle route /api/_test/* aggiunte dal lavoro api-skeleton in PR #17, quindi esamina i test di integrazione che esercitano gli errori di validazione, gli errori interni e la riproduzione dell’idempotenza. Decidi se la rimozione o un controllo dell’ambiente è adatto al piano di transizione; il lavoro è completato quando il comportamento dei test rimane disponibile mentre la produzione non espone più queste route.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

The api-skeleton plan added three stub routes used only by integration tests:

  • POST /api/_test/validation-error — throws ApiValidationError for testing the error mapper
  • POST /api/_test/internal-error — throws a generic Error for testing 500 behavior
  • POST /api/_test/idempotency — tests idempotency key replay

These have { schema: { hide: true } } so they don't appear in the OpenAPI doc, but they're live in production. They should either be removed before cutover or guarded behind NODE_ENV !== 'production'.

Came out of api-skeleton (PR #17), Follow-up item.

Lingua principale
TypeScript
Stelle
1
Fork
1
Merge medio
1g 20h
PR unite (30g)
25

Guida per i contributori

Nessuna guida per i contributori indicizzata per questo repository

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di CodeForPhilly/codeforphilly-ng

Tutte le issue di CodeForPhilly/codeforphilly-ng

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.