"ecc library invalid" when initializing ECPairFactory

Abierto
#21 3 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
32/100
Tipo de issue
Error
Claridad
Necesita aclaración
Estado de actividad
Estancado
Stack tecnológico
javascript, node.js, vite
Área
cryptography

Línea de trabajo

Empieza con la inicialización de ECPairFactory y las aserciones de testecc, especialmente pointFromScalar, sign y signSchnorr. Usa el archivo de reproducción de Caravan enlazado y las pruebas para comparar las compilaciones fallidas de Node y del navegador; se considera terminado cuando la misma biblioteca tiny-secp256k1 se inicializa correctamente sin omitir validaciones.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

There may be some overlap here with another issue I found when trying to initEcc in bitcoinjs-lib with tiny-secp256k1. I put up a fix for that here but even with this I started running into a similar error when trying to init the same library with ecpair. I.e. the same ecc library will init and validate with bitcoinjs-lib but not ECPairFactory. You can see both being initialized in this WIP PR here and the failure in CI here.

In an attempt to narrow down what was causing the failure, I went through each assertion in testecc to find the ones that would fail. By commenting out pointFromScalar, sign, and signSchnorr in the built code in my node_modules, I was able to get the library to verify and use ECPairFactory without problem though it's not clear to me why these are failing (and skipping validations is obviously not ideal).

When consoling out what the assertion was expecting versus what it was getting, this is what I found for pointFromScalar:

actual: <Buffer 03 cd e1 d8 c4 fb e5 ee 21 ff 86 51 5c a7 8d e4 c7 14 d1 cb 8e 60 c0 22 b8 5b 8c cf 6f 83 55 ac b6>
expected: <Buffer 02 b0 7b a9 dc a9 52 3b 7e f4 bd 97 70 3d 43 d2 03 99 eb 69 8e 19 47 04 79 1a 25 ce 77 a4 00 df 99>

I'm getting this in node environments (jest tests specifically) as well as an esbuild/vite build for browser.

There is a branch with code that reproduces this here with the relevant file here and tests here.

Lenguaje dominante
JavaScript
Estrellas
29
Forks
35
Métricas de merge de PR
Sin PR fusionados en 30 d

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de bitcoinjs/ecpair

Todos los issues de bitcoinjs/ecpair

Issues similares

Más issues de JavaScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.