haskell-servant/servant

servant-client requires charset in Accept PlainText

Open

#1 002 ouverte le 2 juil. 2018

Voir sur GitHub
 (7 commentaires) (1 réaction) (0 assignés)Haskell (422 forks)batch import
enhancementhelp wantednewcomer-friendlyservant-api

Métriques du dépôt

Stars
 (1 953 stars)
Métriques de merge PR
 (Merge moyen 23h 44m) (4 PRs mergées en 30 j)

Description

The current implementation is

instance Accept PlainText where
    contentType _ = "text" M.// "plain" M./: ("charset", "utf-8")

Trying to parse text/plain yields UnsupportedContentType text/plain

One API I'm calling does not specify the charset so I can't use PlainText, but this does the job:

data PT
instance Accept PT where
  contentType _ = "text" M.// "plain"

The workaround is simple, but could/should the charset be made optional?

Guide contributeur

Direction de recherche
Le problème concerne l'instance Accept pour PlainText qui nécessite un paramètre charset. Le code pertinent se trouve probablement dans le paquet servant client, peut être dans une définition d'instance Accept. La solution de contournement montre une instance Accept personnalisée sans charset. Les mainteneurs doivent décider de rendre charset facultatif ou d'accepter les deux variantes. Recherchez les commentaires de la discussion et vérifiez s'il y a une PR liée ou une décision.
Stack technique
Aucun
Domaine
backendapi
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
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.
En attente d'un mainteneur
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Haskell basicsservant library
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50