astral-sh/ruff

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

Open

#14,901 建立於 2024年12月10日

在 GitHub 查看
 (3 留言) (0 反應) (0 負責人)Rust (2,088 fork)batch import
help wanted

倉庫指標

Star
 (47,527 star)
PR 合併指標
 (平均合併 3天 8小時) (30 天內合併 573 個 PR)

描述

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.

貢獻者指南