Add new error types for signatures and public keys
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 30/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Cần làm rõ
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- rust
- Lĩnh vực
- blockchain, cryptography
Hướng nghiên cứu
Bắt đầu bằng cách đọc phần xử lý RLP DecoderError và các đường dẫn giải mã cho chữ ký và khóa công khai. Xem lại ví dụ SchnorrSignature và các yêu cầu về tính hợp lệ của điểm trên đường cong elliptic. Công việc được xem là hoàn tất khi các kiểu lỗi mới và hành vi giải mã đối với các điểm không hợp lệ được định nghĩa nhất quán.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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?
- Ngôn ngữ chính
- Rust
- Star
- 36
- Fork
- 11
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của CodeChain-io/foundry
-
Remove informerĐang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 30/100
CodeChain-io/foundry#607 ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 30/100
CodeChain-io/foundry#606 ·
-
Change the return type of execute_transactions to signify it always succeedsCó thể làm lại được @dynaxis đã nhận 2112 ngày trước và không có pull request nào đang mở. Đang mở
CodeChain-io/foundry#605 · 1 reaction · 1 người được giao ·
-
Handle invalid transactions.Đang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 25/100
CodeChain-io/foundry#604 ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
CodeChain-io/foundry#603 ·
Tất cả issue của CodeChain-io/foundry
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
chroma-core/chroma#7879 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
priority middle
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 72/100
KATO-Hiro/AtCoderClans#12838 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
clap_complete env (PowerShell): values after a space don't complete in Windows PowerShell 5.1Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
Maintainer thường phản hồi trong vòng 1 ngày