akka/akka-http

Http: Make PredefinedFromEntityUnmarshallers bail-out after certain nr of bytes read-in

Open

#143 aberto em 8 de set. de 2016

Ver no GitHub
 (9 comments) (0 reactions) (0 assignees)Scala (598 forks)batch import
1 - triagedbughelp wantedt:marshalling

Métricas do repositório

Stars
 (1.311 stars)
Métricas de merge de PR
 (Mesclagem média 1d 10h) (2 fundiu PRs em 30d)

Description

Issue by ktoso Thursday Dec 03, 2015 at 14:52 GMT Originally opened as https://github.com/akka/akka/issues/19080

As discussed in https://github.com/akka/akka/pull/19042#discussion_r46096801

We are currently limiting the total number of bytes in a request body, however the same does not happen for individual Marshallers. This is most noticeably not nice when using FormFields or streaming Source[Element]. We should provide a config value to be provided implicitly after how many bytes such marshaller should bail out – we'd have "per element" limit and "per entire request" limit then.

Guia do colaborador

Direção de pesquisa
Identifique o pipeline de marshalling no Akka HTTP e localize onde os limites de bytes por elemento devem ser aplicados. Procure pela configuração existente para limites totais do corpo da requisição e replique uma lógica similar para marshallers individuais, especialmente para FormFields e fontes de streaming.
Pilha de tecnologia
scala
Domain
backend
Tipo Issue
Funcionalidade
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-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
ScalaAkka
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40