LtbLightning / bdk-rn

Bitcoin Development Kit - React Native Module
MIT License
50 stars 15 forks source link

[Android] - initWallet - descriptor not always defined #75

Open Czino opened 7 months ago

Czino commented 7 months ago

With the new PR, we encounter a new occasional error, it appears often only once or twice and then not again during the session.

The _descriptor[descriptor] is not defined. Possibly a racing condition caused by wrapping every method in a Thread

https://github.com/LtbLightning/bdk-rn/blob/933cee711893e3644b33ef7925ca1996a9b2a0ea/android/src/main/java/io/ltbl/bdkrn/BdkRnModule.kt#L407-L428

Czino commented 7 months ago

The odd thing is, that functions such as newBip84 return the descriptor id for use.

BitcoinZavior commented 7 months ago

Managed to replicate this issue, we should have a fix shortly 👍

BitcoinZavior commented 7 months ago

Managed to replicate this issue, we should have a fix shortly 👍

PR Updated, please see if you still have the issue