wallet sign command returns generic error for decoding failures
Maintainer thường phản hồi trong vòng 2 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 72/100
Hướng nghiên cứu
Bắt đầu từ luồng giải mã PSBT của lệnh bdk-cli wallet sign và theo dõi cách các lỗi giải mã trở thành giá trị BDKCliError. Issue chỉ ra các biến thể hiện có PsbtParseError và DecodeError; hãy kiểm tra ánh xạ lỗi và chạy các bài kiểm thử CLI liên quan. Công việc hoàn tất khi đầu vào PSBT sai định dạng báo lỗi giải mã Base64 mô tả rõ nguyên nhân thay vì lỗi chung chung.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Describe the bug
When the bdk-cli wallet sign command is given a PSBT that cannot be decoded (e.g. malformed base64, truncated bytes, or invalid PSBT structure), the CLI returns a generic error instead of a descriptive one. The underlying BDKCliError already has a dedicated PsbtError and DecodeError variant, but the CLI appears to collapse decoding failures into Error::Generic, which makes it impossible for users to tell whether the problem is their PSBT, their wallet, or the signing step itself.
To Reproduce
- Build
bdk-cliwith any wallet backend. - Create a wallet and obtain a valid PSBT (or use an existing one).
- Corrupt the PSBT string or send a invalid one:
bdk-cli wallet -w test_user1 sign --psbt "cHNidP8BAKkBAAAAAsoKb5SPcP0PpTjvnGo3b4ANmCiirV/br9mIOgkb6DqyAAAAAAD9////"
- Observe the error output.
Expected behavior
The CLI should return a specific, actionable error that identifies the failure as a Base64 decoding problem.for example:
Error: Base64 decode error: Invalid padding
rather than:
Error: Generic error: Invalid padding
Build environment
· BDK-CLI tag/commit: main
· OS+version: macOS 12.7.6 (Monterey)
· Rust/Cargo version: rustc --version output
· Rust/Cargo target: x86_64-apple-darwin
Additional context
The BDKCliError enum already includes a PsbtParseError and DecodeErrorvariant so the fix is simple. I'll like to work on it myself.
- Ngôn ngữ chính
- Rust
- Star
- 142
- Fork
- 103
- Merge trung bình
- 4 ngày 8 giờ
- Pull request đã merge (30 ngày)
- 5
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- 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 bitcoindevkit/bdk-cli
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
bitcoindevkit/bdk-cli#360 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
bitcoindevkit/bdk-cli#358 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Remove "wallet" field from `WalletConfigInner`Có thể đã có người làm @Jennycj đã nhận 6 ngày trước. Đang mởenhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
bitcoindevkit/bdk-cli#332 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 2 ngày
-
enhancement
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
bitcoindevkit/bdk-cli#359 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
`warn!` output is invisible unless `RUST_LOG` is setCó thể đã có người làm @chukwudiikeh đã nhận 4 ngày trước. Đang mở
bitcoindevkit/bdk-cli#352 · 1 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 2 ngày
Tất cả issue của bitcoindevkit/bdk-cli
Issue tương tự
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 88/100
fritzprix/libr-agent#2165 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area:ecosystem bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Chelis-Lang/chelis#3942 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
needs-triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 63/100
Maintainer thường phản hồi trong vòng 1 ngày
-
review-bot
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/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 86/100
Maintainer thường phản hồi trong vòng 1 ngày