haskell-servant/servant

servant-client requires charset in Accept PlainText

Open

#1.002 aberto em 2 de jul. de 2018

Ver no GitHub
 (7 comments) (1 reaction) (0 assignees)Haskell (422 forks)batch import
enhancementhelp wantednewcomer-friendlyservant-api

Métricas do repositório

Stars
 (1.953 stars)
Métricas de merge de PR
 (Mesclagem média 23h 44m) (4 fundiu PRs em 30d)

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?

Guia do colaborador

Direção de pesquisa
O problema é sobre a instância Accept para PlainText exigir um parâmetro charset. O código relevante provavelmente está no pacote servant client, possivelmente em uma definição de instância Accept. A solução alternativa mostra uma instância Accept personalizada sem charset. Os mantenedores precisam decidir se tornam charset opcional ou aceitam ambas as variantes. Pesquise os comentários da discussão e verifique se há algum PR vinculado ou decisão.
Pilha de tecnologia
None
Domain
backendapi
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Aguardando mantenedor
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
Haskell basicsservant library
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50