Feature flag keys with one trailing newline are accepted
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 1/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 88/100
Direzione di ricerca
Leggi il validatore delle chiavi in backend/app/schemas/feature_flag.py e controlla come gestisce i quattro esempi di chiavi descritti nell’issue. Aggiorna la convalida in modo che un carattere di nuova riga finale venga rifiutato con il messaggio esistente per le chiavi non valide. Verifica quindi che le chiavi valide continuino a essere accettate e quelle non valide vengano rifiutate.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
What happens. FeatureFlagCreate accepts a flag key that ends in one newline. FeatureFlagCreate(key="search_v2\n", name="x") is accepted and stores the key as 'search_v2\n'. Keys with a newline anywhere else ("a\nb") and keys with upper case or spaces are refused, as expected.
Why. The key validator in backend/app/schemas/feature_flag.py checks the key with re.match(r"^...$", key). In Python, $ also matches just before a final newline, so one trailing newline passes.
Expected. A key with a trailing newline is refused with the same message as any other invalid key (for example by using re.fullmatch or \Z).
Found while updating the Learn Python book (Hour 8 teaches $ versus \Z). Measured at main 09d815ea with a short script that builds FeatureFlagCreate for four keys.
- Lingua principale
- Python
- Stelle
- 0
- Fork
- 4
- Merge medio
- 6h 43m
- PR unite (30g)
- 414
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
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 Controlvar/controlvar
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
Controlvar/controlvar#1813 ·
I maintainer di solito rispondono entro 1 giorno
-
Dashboard: an archived flag shows Off with an on/off switch the API refusesForse già presa @amarkanday l’ha presa 1 giorno fa. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
Controlvar/controlvar#1748 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Dashboard: the sequential panel's "Ready to stop" banner shows no sample-ratio-mismatch cautionForse già presa @amarkanday l’ha presa 1 giorno fa. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
Controlvar/controlvar#1737 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
dashboard: a split URL draft's variant edit shows generic copy when the API refuses itForse già presa @amarkanday l’ha presa 1 giorno fa. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
Controlvar/controlvar#1562 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 90/100
Controlvar/controlvar#1447 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di Controlvar/controlvar
Issue simili
-
ChatBedrockConverse changes the keys of JSON tool results to camelCase (`order_id` becomes `orderId`)Forse già presa @geyazheng l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
langchain-ai/langchain-aws#1469 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
b4b usability
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
fix(schedule): align list_tasks since cursor filter boundary from strict > to >=Forse già presa @bfs2021 l’ha presa oggi. Apertaagent-ready bug p2 schedule
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
allenporter/supernote#540 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
frazane/scoringrules#164 ·
I maintainer di solito rispondono entro 1 giorno