-
Notifications
You must be signed in to change notification settings - Fork 2k
[FALSE NEGATIVE] "S608 Possible SQL injection" #12044
Copy link
Copy link
Closed
Labels
bugSomething isn't workingSomething isn't workinggood first issueGood for newcomersGood for newcomersruleImplementing or modifying a lint ruleImplementing or modifying a lint rule
Description
Hello.
I'm very exited to migrate our codebase to ruff. However, while doing so I've noticed, that rule S608 works different from the corresponding B608.
It only triggers if SELECT is on the same line.
So SELECT * FROM {foo}.table is an error and
SELECT *
FROM {foo}.table
Is not.
Kind regards, Mikhail
- B608
- S608
- sql injection
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workinggood first issueGood for newcomersGood for newcomersruleImplementing or modifying a lint ruleImplementing or modifying a lint rule