Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Don't use isinstance(..., Sequence) #12123

@squahtx

Description

@squahtx

In a couple of places, we validate incoming lists of strings with isinstance(..., Sequence). Unfortunately this lets plain strings slip through.
We have to make these checks more specific.

Metadata

Metadata

Assignees

Labels

S-TolerableMinor significance, cosmetic issues, low or no impact to users.T-DefectBugs, crashes, hangs, security vulnerabilities, or other reported issues.good first issueGood for newcomers

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions