astral-sh/ruff

Avoid raising S310 if user explicitly checks for URL scheme

Open

#7918 aperta il 11 ott 2023

Vedi su GitHub
 (13 commenti) (11 reazioni) (0 assegnatari)Rust (2088 fork)batch import
help wantedrule

Metriche repository

Star
 (47.527 star)
Metriche merge PR
 (Merge medio 3g 8h) (573 PR mergiate in 30 g)

Descrizione

  • A minimal code snippet that reproduces the bug.
import urllib.request


def foo(url: str) -> None:
    if not url.startswith(("https://", "http://")):
        raise ValueError
    req = urllib.request.Request(
        url,
        headers=headers,
        data=data,
        method=method,
    )
    resp = urllib.request.urlopen(req)

Also it would be great if ruff could recognize asserts (like assert url.startswith(("https://", "http://")))

  • The command you invoked (e.g., ruff /path/to/file.py --fix), ideally including the --isolated flag.

ruff --isolated --select S t.py

  • The current Ruff settings (any relevant sections from your pyproject.toml).

No pyproject.toml

  • The current Ruff version (ruff --version).

ruff 0.0.292

Guida contributor

Direzione di ricerca
Ispeziona l'implementazione della regola S310 nel codice sorgente di Ruff (probabilmente in crates/ruff linter/src/rules/flake8 bandit.rs o simile). Identifica la logica che rileva l'uso di URL e aggiungi un controllo per verificare se è presente un controllo esplicito dello schema URL (come `url.startswith` con https/http) o un'asserzione. Modifica la regola per non sollevare S310 quando tale controllo è presente.
Tech stack
pythonrust
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
Pythonbasic Rust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60