python-bitcoinlib icon indicating copy to clipboard operation
python-bitcoinlib copied to clipboard

VerifyScript not upgraded for segwit

Open kanzure opened this issue 5 years ago • 3 comments
trafficstars

See upstream updates for VerifyScript to handle witness validation: https://github.com/bitcoin/bitcoin/blob/a654626f076a72416a3d354218d7107571d6caaf/src/script/interpreter.cpp#L1472

kanzure avatar Jan 19 '20 20:01 kanzure

python-bitcointx's scripteval.py (https://github.com/Simplexum/python-bitcointx/blob/master/bitcointx/core/scripteval.py) supports segwit, but I feel it will require some work to backport.

Easier way might be to backport an interface to bitcoinconsensus library: https://github.com/Simplexum/python-bitcointx/blob/master/bitcointx/core/bitcoinconsensus.py -- there's not that much code.

dgpv avatar Jan 19 '20 20:01 dgpv

Any idea when this will be implemented?

farahats9 avatar Nov 07 '23 01:11 farahats9

On Mon, Nov 06, 2023 at 05:26:09PM -0800, farahats9 wrote:

Any idea when this will be implemented?

Probably never. What I should be doing is removing VerifyScript as there's no way that it'll be consensus-compatible with Bitcoin Core.

petertodd avatar Nov 07 '23 22:11 petertodd