bitcoindevkit / bitcoindevkit/bdk-ffi

fix!: return errors from Wallet.peek_address

Open
#1,105 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
127
Forks
81
Avg merge
1d 2h
Merged PRs (30d)
5

Description

Change `Wallet.peek_address` to return errors for out of range derivation indexes and descriptors that cannot produce an address (instead of allowing upstream `expect()` calls to panic)

Contributor guide

No contributing guide indexed for this repository

Research direction

Start at the Wallet.peek_address entry point and trace how out-of-range derivation indexes and descriptors that cannot produce an address are handled. Done means both cases return errors to callers instead of reaching upstream expect() calls and panicking; run the relevant existing tests or add coverage for these two cases.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
api
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
68/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.