spesmilo / electrum

Electrum Bitcoin Wallet
https://electrum.org
MIT License
7.5k stars 3.1k forks source link

Trezor: regtest bech32 addresses cause cryptic error #5051

Open afk11 opened 5 years ago

afk11 commented 5 years ago

It seems Bitcoin Core + electrum changed the regtest bech32 prefix a while ago to 'bcrt1'. At the moment if you try paying to a P2WPKH or P2WSH address, on regtest, with a trezor, you get a cryptic 'Failed to compile output' error.

I checked https://github.com/trezor/trezor-common/tree/master/defs/bitcoin and sure enough there was no definition for regtest, so maybe it worked before the prefixes changed.. but it's broken now.

I'm hoping we can find a way to indicate this combination is bad, so people don't lose much time to it in future.

matejcik commented 5 years ago

regtest is now supported in Trezor FW as a separate "coin" (similar to Testnet), so all Electrum needs to do is indicate the right coin name