astral-sh/ruff

Avoid raising S310 if user explicitly checks for URL scheme

Open

#7.918 aberto em 11 de out. de 2023

Ver no GitHub
 (13 comments) (11 reactions) (0 assignees)Rust (2.088 forks)batch import
help wantedrule

Métricas do repositório

Stars
 (47.527 stars)
Métricas de merge de PR
 (Mesclagem média 3d 8h) (573 fundiu PRs em 30d)

Description

  • 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

Guia do colaborador

Direção de pesquisa
Inspecione a implementação da regra S310 no código fonte do Ruff (provavelmente em crates/ruff linter/src/rules/flake8 bandit.rs ou similar). Identifique a lógica que detecta o uso de URL e adicione uma verificação para ver se há uma verificação explícita de esquema de URL (como `url.startswith` com https/http) ou uma asserção. Modifique a regra para não gerar S310 quando tal verificação estiver presente.
Pilha de tecnologia
pythonrust
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
Pythonbasic Rust
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60