valyala/fasthttp

Chunked response with trailer result in "cannot find crlf at the end of chunk" error

Fermée

#1 078 ouverte le 21 août 2021

Voir sur GitHub
 (1 commentaire) (1 réaction) (0 personne assignée)Go (1 755 forks)batch import
help wanted

Métriques du dépôt

Stars
 (21 741 étoiles)
Métriques de merge PR
 (Merge moyen 3j) (44 PRs mergées en 30 j)

Description

Hello,

While using fasthttp.Client to talk to a server that produce chunked response with trailer headers I see "cannot find crlf at the end of chunk" error. I suspects that this happens because fasthttp's implementation does not expect to see trailer headers after the last chunk, which is not according to chunked transfer coding spec.

This can ideally be fixed by adding https://github.com/valyala/fasthttp/issues/210. Otherwise readBodyChunked need to be mindful of trailers.

Let me know if I should produce an example.

Regards.

Guide contributeur

Direction de recherche
Examinez la RFC sur l'encodage de transfert en morceaux (chunked) et la fonction `readBodyChunked` dans `http.go` ou un fichier similaire. Étudiez comment les en têtes de fin (trailers) sont actuellement gérés et si la correction de l'issue #210 (prise en charge des trailers) peut être intégrée. Envisagez d'ajouter une logique pour analyser les en têtes de fin après le morceau de terminaison.
Stack technique
go
Domaine
api
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.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
HTTP chunked transfer encodingGo language basicsfasthttp library structure
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60