semaphoreio/semaphore

Add missing `Content-Type` headers in public api

Aperta

#372 aperta il 9 giu 2025

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (2 assegnatari)Elixir (77 fork)batch import
enhancementgood first issue

Metriche repository

Star
 (1519 stelle)
Metriche merge PR
 (Merge medio 6g 15h) (40 PR mergiate in 30 g)

Descrizione

Is your improvement proposal related to a problem? Please describe.

Currently, only some responses from the public API have a Content-Type header set. This can lead to issues when the requester expects a specific content type and it is not provided.

Describe the solution you'd like

Add Content-Type headers to all responses from the public API, ensuring that they are set to the appropriate type based on the content being returned (e.g., application/json for JSON responses).

Describe alternatives you've considered

No response

Additional context

Guida contributor

Direzione di ricerca
Esamina il router dell'API pubblica in public api/v1alpha/lib/pipelines api/router.ex e aggiungi intestazioni Content Type a tutte le risposte, assicurandoti che corrispondano al contenuto della risposta (ad esempio, application/json per JSON).
Tech stack
elixir
Dominio
backend api design
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Elixir
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75