valyala/fasthttp

Request should not fail on small ReadBufferSize

Fermée

#1 873 ouverte le 24 sept. 2024

Voir sur GitHub
 (4 commentaires) (0 réaction) (0 personne assignée)Go (1 755 forks)batch import
help wantedwontfix

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

Can we not fail request if the req/resp header size is greater than the read buffer size.

I can see a bufio bug was mentioned for tryRead function but looks like it's resolved now. Can we have the above mentioned behaviour?

Also, some callback could also be configured to inform that buffer size was low. Library users could emit metrics and increase the buffer size appropriately to have best balance between memory usage and CPU usage.

@valyala Could you review the request?

Guide contributeur

Direction de recherche
Tout d'abord, lisez le fichier header.go de fasthttp, en particulier la fonction tryRead à la ligne 2128 et la logique de tampon environnante. Comprenez comment ReadBufferSize affecte l'analyse des requêtes. Le problème fait référence à un bug Go (golang/go#14121) déjà résolu ; étudiez si cette résolution peut être appliquée à fasthttp. En outre, envisagez de mettre en œuvre un mécanisme de rappel pour informer les utilisateurs lorsque la taille du tampon est faible, leur permettant ainsi de s'adapter. Vérifiez les commentaires existants pour tout retour des mainteneurs ou contexte supplémentaire.
Stack technique
gorest api
Domaine
backendperformance
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.
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.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Plutôt claire
Prérequis
Understanding of GoFamiliarity with fasthttp's bufferingReading header.go file
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
30