Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Override Script Eval Context for Elements #1009

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Kukks
Copy link
Contributor

@Kukks Kukks commented May 27, 2021

Adds: CHECKSIGFROMSTACK, CHECKSIGFROMSTACKVERIFY
Re-enables: CAT

I haven't figured out a good way to plug this into the system painlessly though. I think I need to add CreateScriptEvaluationContext to ConsensusFactory but that requires I need to do method signature changes across the lib

Adds: CHECKSIGFROMSTACK, CHECKSIGFROMSTACKVERIFY
Re-enables: CAT

I haven't figured out a good way to plug this into the system painlessly though. I think I need to add `CreateScriptEvaluationContext` to `ConsensusFactory` but that requires I need to do method signature changes across the lib
@NicolasDorier
Copy link
Collaborator

@Kukks do we need this btw? I am not sure it makes sense to validate the scripts with NBitcoin, even if we support it.
Probably better to rely on Bitcoin Core for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants