HomePhabricator

Schnorr signature activation

Description

Schnorr signature activation

Summary:
Includes python activation tests based on replay protection activation.

Features:

  • Tests that when upgrade / downgrade happens, newly-invalid transactions are dumped from mempool (but not valid ones).
  • Don't ban peers who send a tx that would be valid under the mandatory flags on other side of upgrade/downgrade.
  • Tests of 64-byte DER signatures

Test Plan:
run functional tests
uncomment fakeDER64 and patch VerifySignature (see comment in file about fakeDER64)
run abc-schnorr-activation.py again

(I have also run these tests with the various critical bits of mainline code cancelled out, and verified the tests fail as expected.)

Reviewers: #bitcoin_abc, deadalnix, jasonbcox, Fabien

Reviewed By: #bitcoin_abc, deadalnix, jasonbcox, Fabien

Subscribers: Fabien, jasonbcox, florian, Mengerian, teamcity, schancel

Maniphest Tasks: T527

Differential Revision: https://reviews.bitcoinabc.org/D2483

Details

Provenance
Mark Lundeberg <36528214+markblundeberg@users.noreply.github.com>Authored on Feb 12 2019, 15:59
dagurvalPushed on Feb 15 2019, 13:14
Reviewer
Restricted Project
Differential Revision
D2483: Schnorr signature activation
Parents
rSTAGING0d4393e4b638: BIP32 extra test vector
Branches
Unknown
Tags
Unknown
References
tag: phabricator/base/7358, tag: phabricator/base/7357, tag: phabricator/base/7356

Event Timeline

Mark Lundeberg <36528214+markblundeberg@users.noreply.github.com> committed rSTAGING6bb69585f326: Schnorr signature activation (authored by Mark Lundeberg <36528214+markblundeberg@users.noreply.github.com>).Feb 15 2019, 03:19