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

Sign with proper forkId #4

Merged
merged 1 commit into from
Mar 22, 2018
Merged

Sign with proper forkId #4

merged 1 commit into from
Mar 22, 2018

Conversation

lnovy
Copy link

@lnovy lnovy commented Mar 22, 2018

I don't even... this crap took me like 5 hours...

I don't even... this crap took me like 5 hours...
Copy link

@martin-key martin-key left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

. Yeah, I got this wrong. Please rename the PR and I will merge this.

@lnovy lnovy changed the title Jesus fscking crust Sign with proper forkId Mar 22, 2018
@lnovy
Copy link
Author

lnovy commented Mar 22, 2018

Zo wish...

@lnovy
Copy link
Author

lnovy commented Mar 22, 2018

Also your transaction loading thingy gets the txout hash mirror-reversed... Than it works (in some basic cases like signing with no more than one key)

@martin-key
Copy link

martin-key commented Mar 22, 2018 via email

@martin-key martin-key merged commit e02fa93 into poolgold:master Mar 22, 2018
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