semaphoreio/semaphore

Add missing `Content-Type` headers in public api

Ouverte

#372 ouverte le 9 juin 2025

Voir sur GitHubGuide contributeur
 (3 commentaires) (0 réaction) (2 personnes assignées)Elixir (77 forks)batch import
enhancementgood first issue

Métriques du dépôt

Stars
 (1 519 étoiles)
Métriques de merge PR
 (Merge moyen 6j 15h) (40 PRs mergées en 30 j)

Description

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

Guide contributeur

Direction de recherche
Examinez le routeur de l'API publique dans public api/v1alpha/lib/pipelines api/router.ex et ajoutez des en têtes Content Type à toutes les réponses, en veillant à ce qu'ils correspondent au contenu de la réponse (par exemple, application/json pour JSON).
Stack technique
elixir
Domaine
backend api design
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Elixir
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75