Add new error types for signatures and public keys
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 30/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Necesita aclaración
- Estado de actividad
- Estancado
- Stack tecnológico
- rust
- Área
- blockchain, cryptography
Línea de trabajo
Comienza leyendo el manejo de RLP DecoderError y las rutas de decodificación para firmas y claves públicas. Revisa el ejemplo SchnorrSignature y los requisitos de validez de los puntos de la curva elíptica. Se considera completado cuando los nuevos tipos de error y el comportamiento de decodificación para puntos no válidos estén definidos de forma coherente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Currently, all signatures and public keys are just wrapper structs of bytes.
For better performance and consistency, they should be wrappers of some types representing points in elliptic curves.
Verifying that a signature instance refers to a valid point in an elliptic curve should be done at the moment of decoding, not at the moment of signature verification.
For example, Schnorr signature defined as pub struct SchnorrSignature([u8; 64]); should be changed to pub struct SchnorrSignature(G1).
I believe we need a different way to handle decoding errors. i.e. decoding of signatures and public keys should fail if the RLP representation is correct, but the decoded bytes do not refer to a valid point in the elliptic curve.
Currently, RLP DecoderError does not support such invalidity.
Do you have any nice idea to handle this error?
- Lenguaje dominante
- Rust
- Estrellas
- 36
- Forks
- 11
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de CodeChain-io/foundry
-
Remove informerAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 30/100
CodeChain-io/foundry#607 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
CodeChain-io/foundry#606 ·
-
Change the return type of execute_transactions to signify it always succeedsQuizá libre de nuevo @dynaxis la tomó hace 2108 días y no hay ningún pull request abierto. Abierto
CodeChain-io/foundry#605 · 1 reacción · 1 asignado ·
-
Handle invalid transactions.Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
CodeChain-io/foundry#604 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
CodeChain-io/foundry#603 ·
Todos los issues de CodeChain-io/foundry
Issues similares
-
Signals (Failure Detector): a tool call and its own execution are reported as a repeated callAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
check: a failed re-read of the model file before binding is labelled E_THETA_LEVEL_BINDING on [parameters]Posiblemente ocupada @TeunP la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 80/100
Devolutions/picky-rs#546 · 1 comentario ·
Los mantenedores suelen responder en 3 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día