Missing safety comment: which variants are upheld?
Les mainteneurs répondent en général sous 2 jours
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 48/100
- Type d'issue
- Documentation
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- rust
- Domaine
- networking
Piste de recherche
Commencez par src/proto/h2/server.rs vers la ligne 478 et examinez UpgradedSendStream::new. Lisez d’abord le lien vers la documentation de Rust API Guidelines, puis déterminez quelles invariants le commentaire de sécurité doit décrire et quelles invariants le constructeur exige. Le travail est terminé lorsque les deux emplacements documentent clairement ces invariants.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Per Rust API Guidelines, unsafe functions should be documented with invariants:
https://rust-lang.github.io/api-guidelines/documentation.html
During review of the source, it was pointed out there is no safety comment here that describes what invariants are upheld:
https://github.com/hyperium/hyper/blob/90eb95f62a32981cb662b0f750027231d8a2586b/src/proto/h2/server.rs#L478
Also UpgradedSendStream::new does not state which invariants need to be upheld.
Would it be possible to add this?
- Langage dominant
- Rust
- Étoiles
- 16.3k
- Forks
- 1.8k
- Merge moyen
- 4 j 10 h
- PR mergées (30 j)
- 21
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
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/hyper
-
Publicly reexport the http crateOuverteC-feature
Difficulté 1/5 Moins d'une heure Accessibilité débutants 65/100
hyperium/hyper#2652 · 4 réactions ·
Les mainteneurs répondent en général sous 2 jours
-
Write a HIP for `Body::poll_progress()`Peut-être pris @cratelyn l’a pris il y a 1 jour. OuverteA-body B-rfc C-feature
hyperium/hyper#4228 · 1 personne assignée ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 3/5 1-2 jours Accessibilité débutants 74/100
hyperium/hyper#4211 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
-
Upgraded HTTP/2 CONNECT streams cannot be reset, so a failed tunnel looks like a clean closePeut-être pris @jeremyjpj0916 l’a pris il y a 10 jours. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 55/100
Les mainteneurs répondent en général sous 2 jours
-
hyper-util legacy client: an HTTP/1 request can hang forever when the connection closes while the request is being queuedPeut-être pris @akasakariko l’a pris il y a 3 jours. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 62/100
hyperium/hyper#4202 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
Toutes les issues de hyperium/hyper
Issues similaires
-
test(executor_l0): assert execute() TaskOutcome, not only bus events / 断言 execute() 返回的 TaskOutcomeOuvertetype:debt
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
skaiy/wild_agentos#425 ·
Les mainteneurs répondent en général sous 1 jour
-
Default-import note suggests `import * as process` for velt:process, which does not name the builtinOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
bug ticket
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
cratestack/cratestack#1154 ·
Les mainteneurs répondent en général sous 1 jour
-
status:needs-triage
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
agentic-os-org/ANOLISA#6742 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour