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

Aberta Para iniciantes
  • Algo errado?
#2,347 1 comentário 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
1/5
Tempo estimado
Menos de uma hora
Facilidade para iniciantes
88/100
Tipo de issue
Documentação
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
c
Domínio
documentation

Direção de pesquisa

Leia o cabeçalho em src/hash.c e o comentário acima de register_hash_builtins em src/builtins.c; compare-os com os nomes registrados listados na issue. Execute git grep -n -i hmac -- src/ lib/ docs/ para encontrar outras promessas de um builtin HMAC. A tarefa estará concluída quando os comentários mencionarem apenas builtins registrados e todas as outras promessas encontradas forem corrigidas.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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

Linguagem predominante
C
Estrelas
3
Forks
7
Merge médio
6h 32min
PRs com merge (30d)
285

Preparar o ambiente

Abrir no Codespaces

Inicia o contêiner de desenvolvimento do projeto no navegador, com a sua própria conta do GitHub.

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de InauguralSystems/EigenScript

Todas as issues de InauguralSystems/EigenScript

Issues semelhantes

Mais issues de C

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.