Header fields should be case insensitive (RFC 7230 §3.2, RFC 9110 §5.1)
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 35/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- rust
- Domaine
- api, networking
Piste de recherche
Commencez par examiner le comportement de HeaderMap::get et HeaderMap::insert conjointement avec HeaderName::from_static, en utilisant comme reproduction les exemples de l’issue sensibles à la casse. La tâche est terminée lorsque la récupération des noms d’en-tête accepte des casses différentes, que from_static ne provoque plus de panique pour les noms de champ valides et que la casse d’origine est conservée.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
I'm working with http::HeaderMap and noticed that HeaderMap::get, HeaderMap::insert, and HeaderName::from_static all seem to be case sensitive (in fact, the latter panics:
HeaderName::from_static("Content-Type")
panics with:
index out of bounds: the len is 0 but the index is 0
while:
HeaderName::from_static("content-type")
does not.
This goes against the HTTP spec requiring case-insensitive handling of header values:
- RFC 7230 §3.2
Each header field consists of a case-insensitive field name followed by a colon (":"), optional leading whitespace, the field value, and optional trailing whitespace.
- RFC 9110 §5.1
Field names are case-insensitive...
IMO, this is distinct from #670, and in fact case-insensitive keys should solve that issue as well, while not requiring the original cases to be changed.
It looks like crates like case_insensitive_hashmap and unicase are examples of case-insensitive String handling.
Is there any way to just make the HeaderMap case-insensitive for retrieval? That would preserve original case as well as allow any case for HeaderMap::get to meet the spec and prevent errors.
- Langage dominant
- Rust
- Étoiles
- 1.4k
- Forks
- 385
- Merge moyen
- 1 j 5 h
- PR mergées (30 j)
- 2
Préparer son environnement
Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de hyperium/http
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
-
Difficulté 3/5 1-2 jours Accessibilité débutants 52/100
-
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
-
Difficulté 3/5 1-2 jours Accessibilité débutants 62/100
-
Difficulté 3/5 1-2 jours Accessibilité débutants 58/100
Toutes les issues de hyperium/http
Issues similaires
-
agent duplicate providers tool triaged
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
Les mainteneurs répondent en général sous 1 jour
-
Text overflow in import boxOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
Les mainteneurs répondent en général sous 1 jour
-
founders_reward panics on a configured Testnet when the slow-start subsidy is not a multiple of 5Ouverteexternal-contribution
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
ZcashFoundation/zebra#11792 ·
Les mainteneurs répondent en général sous 1 jour
-
content:edit documentation effort:high
Difficulté 1/5 1-3 heures Accessibilité débutants 67/100
cloudflare/cloudflare-docs#34428 ·
Les mainteneurs répondent en général sous 1 jour
-
area/cli bug
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100