Feature flag keys with one trailing newline are accepted
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 1/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 88/100
Rechercherichtung
Lies den Schlüsselvalidator in backend/app/schemas/feature_flag.py und prüfe, wie er die vier im Issue beschriebenen Schlüsselbeispiele behandelt. Aktualisiere die Validierung, sodass ein abschließender Zeilenumbruch mit der vorhandenen Meldung für ungültige Schlüssel abgelehnt wird. Überprüfe anschließend, dass gültige Schlüssel weiterhin akzeptiert und ungültige Schlüssel abgelehnt werden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- Python
- Sterne
- 0
- Forks
- 4
- Ø Merge
- 6 Std. 43 Min.
- Gemergte PRs (30 T.)
- 414
Entwicklungsumgebung
- Enthält ein Dockerfile oder eine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus Controlvar/controlvar
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
Controlvar/controlvar#1813 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Dashboard: an archived flag shows Off with an on/off switch the API refusesEvtl. vergeben @amarkanday hat das vor 2 Tagen übernommen. Offenbug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Controlvar/controlvar#1748 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Dashboard: the sequential panel's "Ready to stop" banner shows no sample-ratio-mismatch cautionEvtl. vergeben @amarkanday hat das vor 2 Tagen übernommen. Offenbug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Controlvar/controlvar#1737 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
dashboard: a split URL draft's variant edit shows generic copy when the API refuses itEvtl. vergeben @amarkanday hat das vor 2 Tagen übernommen. Offenbug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
Controlvar/controlvar#1562 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 90/100
Controlvar/controlvar#1447 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in Controlvar/controlvar
Ähnliche Issues
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
igormcsouza/tether#278 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 91/100
aws-samples/sample-per-model-bedrock#803 ·
Maintainer antworten meist innerhalb von 1 Tag
-
documentation p1
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
huggingface/accelerate#4511 ·
Maintainer antworten meist innerhalb von 1 Tag
-
follow-up
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 1 Tag