4 commentaires (4 commentaires)2 réactions (2 réactions)0 assignés (0 assignés)Rust84 934 stars (84 934 stars)3 111 forks (3 111 forks)batch import
bughelp wanted
Description
e.g., https://github.com/astral-sh/uv/issues/16233
We should not be retrying that failed request.
Guide contributeur
- Stack technique
- rust
- Domaine
- securityapi
- 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.
- 1-3 hours
- Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
- fresh
- ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
- mostly clear
- Prérequis
- basic Rustunderstanding of HTTP retry logic
- Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
- 40
- Direction de recherche
- The issue requests that SSL certificate errors should not be retried when making HTTP requests. Start by locating the retry logic in the codebase, likely in the HTTP client module (e.g., src/**/http.rs or similar). Look for where errors are caught and retries are triggered. Identify how SSL errors are represented (e.g., via the `rustls` or `native tls` crate) and ensure that such errors skip the retry loop. No linked PRs exist; refer to the referenced issue ( #16233 ) for context. After implementing, verify that the behavior matches expectations by simulating an SSL error.