Cookie path can inject extra Set-Cookie attributes via `;`
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 82/100
Direzione di ricerca
Inizia da MaxLib.WebServer/HttpCookie.cs:144 e verifica come HttpCookie.Cookie.ToString() esegue il rendering di Path nella riga Set-Cookie. Mantieni la sanitizzazione locale al rendering del cookie, quindi verifica che i punti e virgola e i caratteri di controllo non possano aggiungere attributi, mentre i normali percorsi dei cookie continuino a essere renderizzati correttamente.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
HttpCookie.Cookie.ToString() only strips CR/LF from Path. new HttpCookie.Cookie(..., path: "/x;Domain=evil.example;SameSite=None") writes the extra Domain and SameSite attributes into the same Set-Cookie line. It matters for applications that build the path from request-influenced text, since the attacker can then widen the cookie scope or downgrade SameSite.
Location: MaxLib.WebServer/HttpCookie.cs:144
Fix: Strip ; (and other control characters) from Path when rendering it. Keep it local to the cookie code, since ; is legitimate in other headers.
- Lingua principale
- C#
- Stelle
- 1
- Fork
- 3
- Merge medio
- 1g 9h
- PR unite (30g)
- 4
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 Garados007/MaxLib.WebServer
-
Reason-phrase table has misspelled entries (`Unathorized`, `Switching Protcols`, `Multiple Choises`)Forse già presa @Jah-yee l’ha presa 13 giorni fa. Apertabug severity:low
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
-
documentation good first issue severity:low
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
-
bug severity:low
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
-
bug security severity:medium
Difficoltà 2/5 1-3 ore Idoneità per principianti 80/100
-
bug severity:low
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
Tutte le issue di Garados007/MaxLib.WebServer
Issue simili
-
area-multitenancy area-security p1-recommended priority-high technical-debt
Difficoltà 2/5 Meno di un'ora Idoneità per principianti 75/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
-
kind/enhancement Provider Upgrade pulumi/pulumi-azure R1 Task
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
pulumi/pulumi-azure#3669 ·
-
[BUG] FromCString causes 1 byte OOB read in native codeForse già presa @mattleibow l’ha presa 1 giorno fa. Apertatype/bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
mono/SkiaSharp#5426 · 1 reazione · 2 assegnatari ·
I maintainer di solito rispondono entro 1 giorno
-
webclient does not startForse già presa @PawelTroka l’ha presa 1 giorno fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
PawelTroka/Computator.NET#257 ·
I maintainer di solito rispondono entro 1 giorno