secure_equals raises on a non-string, but BUILTINS.md says non-strings return false
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 67/100
- Issue-Typ
- Bug
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- c
- Bereich
- documentation, testing
Rechercherichtung
Read docs/BUILTINS.md at line 123 and inspect the runtime implementation and existing tests for secure_equals to decide which behavior is intended. Update the documentation or runtime so they agree, add a test pinning the chosen behavior, and run the relevant tests.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
docs/BUILTINS.md:123 documents secure_equals as "Non-strings → false". On main (d3ec498d), print of (secure_equals of [null, "abc"]) raises secure_equals: expected two strings.
Raising is arguably the better contract (a null secret is a caller bug, and fail-loud is the house rule), so this is probably a docs fix, not a runtime change. Decide which, then make them agree.
Found while reviewing the #2249 fix: lib/auth.eigs only ever passes strings (_extract_bearer returns "" when the header is missing), so auth is not affected.
Done when
-
docs/BUILTINS.mdand the runtime agree on non-string arguments, with a test pinning the chosen behaviour.
— Claude DevBox
- Vorherrschende Sprache
- C
- Sterne
- 3
- Forks
- 7
- Ø Merge
- 6 Std. 32 Min.
- Gemergte PRs (30 T.)
- 285
Entwicklungsumgebung
Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.
- 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 InauguralSystems/EigenScript
-
good first issue needs-triage
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 88/100
InauguralSystems/EigenScript#2347 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
needs-triage
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 82/100
InauguralSystems/EigenScript#2335 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
area:embed kind:bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
InauguralSystems/EigenScript#2311 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
area:gates kind:gate-defect
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
InauguralSystems/EigenScript#2305 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
area:ci bug found-by:duet-audit
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
InauguralSystems/EigenScript#2283 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in InauguralSystems/EigenScript
Ähnliche Issues
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
FujiNetWIFI/fujinet-config#379 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Ship /Local/Applications, the Local domain's Applications folderEvtl. vergeben @pkgdemon hat das heute übernommen. Offenarea:base enhancement
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 94/100
nextbsd/nextbsd-userland#612 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
zephyrproject-rtos/zephyr#125543 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
colbymchenry/codegraph#3277 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100