fix(hooks): security guard misses raw SQL built on an earlier line or across lines
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 57/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- javascript
Direzione di ricerca
Read the hook implementation that checks added lines and its existing tests; the payload does not name their paths. Trace how it analyzes added blocks and calls to FromSqlRaw, ExecuteSqlRaw, and SqlQueryRaw, then run the hook tests. Done means both multiline call shapes and interpolated, concatenated, or formatted string variables are flagged, with large file checks staying under the 10 second timeout.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
What
The security guard checks each added line on its own, so var sql = $"... {id}"; db.Database.ExecuteSqlRaw(sql); and a FromSqlRaw( call split over several lines pass without a warning.
Why
Raw SQL built from strings is the guard's most valuable .NET rule, and both shapes are common.
Acceptance criteria
- A string variable built with interpolation, concatenation or
string.Formatand later passed toFromSqlRaw,ExecuteSqlRaworSqlQueryRawin the same added block is flagged. - A call whose argument starts on the next line is flagged.
- The hook still finishes well under its 10 s timeout on large files, with tests for both shapes.
- Lingua principale
- JavaScript
- Stelle
- 0
- Fork
- 0
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di devmaxxx/stackwise
-
feat: add a plugin iconApertaenhancement good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
enhancement
Difficoltà 3/5 1-2 giorni Idoneità per principianti 55/100
-
enhancement
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
-
bug eval
Difficoltà 3/5 1-2 giorni Idoneità per principianti 66/100
-
bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 55/100
Tutte le issue di devmaxxx/stackwise
Issue simili
-
Clinic sharing: fast sync refusal message suggests Delete Data, which only developer builds haveApertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
avniproject/avni-client#2228 ·
-
bug confirmed issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
open-webui/open-webui#33196 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
`printf "%o"` prints decimalAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
fable-compiler/Fable#5157 ·
I maintainer di solito rispondono entro 1 giorno
-
New tool: PaperBananaForse già presa @github-actions l’ha presa oggi. Apertasubmission triage
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 75/100
kausalflow/tools#243 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
area: apple-music area: build-release bug effort: small priority: high
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
ChristianTabor/Argon#32 ·
I maintainer di solito rispondono entro 1 giorno