astral-sh/ruff

flake8-bandit import check should not trigger on TYPE_CHECKING imports or classes not in defusedxml

Open

#14.901 aperta il 10 dic 2024

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)Rust (2088 fork)batch import
help wanted

Metriche repository

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

Descrizione

The following code triggers S408 ("xml.dom.minidom is vulnerable to XML attacks"):

from typing import TYPE_CHECKING

if TYPE_CHECKING:
    from xml.dom.minidom import Element

As far as I know, defusedxml, which this rule suggests as an alternative, does not supply alternative implementations for most of the types, only of some functions. In other words, I have to import types like these for the standard library; there is no defusedxml alternative.

So in order to signal to Ruff that "this is fine"™, I've tried moving the import to TYPE_CHECKING, but still received the same error.

This probably applies to other rules in the S4xx range, too.

Guida contributor

Direzione di ricerca
Investigare come la regola S408 di Ruff gestisce le importazioni TYPE CHECKING e verificare che defusedxml non fornisca alternative di tipo per xml.dom.minidom. Aggiornare la regola per saltare le importazioni sotto TYPE CHECKING.
Tech stack
python
Dominio
tooling
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonRuffTYPE CHECKING
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65