HeaderValue::to_str returns an error for valid unicode strings
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 45/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- rust
- Domaine
- api, networking
Piste de recherche
Commencez par localiser HeaderValue::to_str et la définition de ToStrError, puis examinez les tests existants pour la conversion des valeurs d’en-tête en chaînes. Vérifiez comment la création de HeaderValue valide les octets avant de modifier le comportement de conversion. Le travail est terminé lorsque l’UTF-8 valide, y compris Unicode et les tabulations, est renvoyé, tandis que l’UTF-8 invalide produit toujours une erreur et que le nom ToStrError reste compatible.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
HeaderValue::to_str() returns an error if the header value contains any byte that's not "visible ASCII" (though this is misleading as \t is legal even though it's not visible). This is weird, it should just return a string if it contains a valid string regardless of the contents. We already know it doesn't contain illegal bytes (as that's checked upon creation of HeaderValue) so we don't need to check again, and there's no reason to disallow unicode characters when getting the string representation.
I think the right thing here is to just replace the implementation with std::str::from_utf8(), and replace ToStrError with a pub use std::str::Utf8Error as ToStrError for backwards compatibility.
Also see #519.
- 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
-
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
-
area/cli bug
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 67/100
bherbruck/solvecraft#2167 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
bitcoindevkit/bdk-cli#357 ·
Les mainteneurs répondent en général sous 3 jours