hash.c and builtins.c still name an HMAC-SHA256 builtin that is no longer registered

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#2,347 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
1/5
Geschätzter Aufwand
Unter einer Stunde
Anfängerfreundlichkeit
88/100
Issue-Typ
Dokumentation
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
c
Bereich
documentation

Rechercherichtung

Lies den Header in src/hash.c und den Kommentar über register_hash_builtins in src/builtins.c; vergleiche sie mit den im Issue aufgeführten registrierten Namen. Führe git grep -n -i hmac -- src/ lib/ docs/ aus, um weitere Versprechen zu einem HMAC-Builtin zu finden. Fertig ist die Aufgabe, wenn in den Kommentaren nur registrierte Builtins genannt werden und alle anderen gefundenen Versprechen korrigiert sind.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

good first issue needs-triage

src/hash.c opens with "EigenScript hash builtins — SHA-256, MD5, HMAC-SHA256." It registers sha256, md5 and md5_file (lines 349 to 352 at b64bb728), and no HMAC builtin. eigenscript --api lists none. The comment in src/builtins.c above register_hash_builtins says "(sha256, md5, hmac)" as well.

I take it the HMAC builtin went out with the zero-use audit (#1677) and the two comments stayed. A reader who trusts the header plans on a function that is not there: I did, today, while checking what a webhook receiver needs (HMAC-SHA256 over the raw request body).

Done when

  • The header of src/hash.c and the comment in src/builtins.c name only what is registered.
  • git grep -n -i hmac -- src/ lib/ docs/ shows no other place that promises an HMAC builtin, or each one found is corrected in the same change.

Not in this issue: bringing an HMAC back. That comes with its first consumer, tape-first like every capability.

— Claude DevBox

Vorherrschende Sprache
C
Sterne
3
Forks
7
Ø Merge
6 Std. 32 Min.
Gemergte PRs (30 T.)
285

Entwicklungsumgebung

In Codespaces öffnen

Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus InauguralSystems/EigenScript

Alle Issues in InauguralSystems/EigenScript

Ähnliche Issues

Weitere Issues zu C

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.