electrum

Electrum Bitcoin wallet
git clone https://git.parazyd.org/electrum
Log | Files | Refs | Submodules

commit aaa67adb5ab421c75ebe8cae6a4989ab88d62008
parent eca8d13e37920ffef999aab01f9c60a0a8dce9f6
Author: SomberNight <somber.night@protonmail.com>
Date:   Fri, 13 Apr 2018 18:47:47 +0200

fixes for make_commitment, but still incorrect destination address (csv arg?)

Diffstat:
Mlib/lnbase.py | 16+++++++++-------
Mlib/tests/test_lnbase.py | 15++++++++++-----
2 files changed, 19 insertions(+), 12 deletions(-)

diff --git a/lib/lnbase.py b/lib/lnbase.py @@ -21,12 +21,12 @@ import hashlib import hmac import cryptography.hazmat.primitives.ciphers.aead as AEAD -from .bitcoin import public_key_from_private_key, ser_to_point, point_to_ser, string_to_number, deserialize_privkey, EC_KEY, rev_hex, int_to_hex +from .bitcoin import public_key_from_private_key, ser_to_point, point_to_ser, string_to_number, deserialize_privkey, EC_KEY, rev_hex, int_to_hex, push_script, var_int, op_push from . import bitcoin from .constants import set_testnet, set_simnet from . import constants from . import transaction -from .util import PrintError, bh2u, print_error +from .util import PrintError, bh2u, print_error, bfh from .wallet import Wallet from .storage import WalletStorage from .transaction import opcodes, Transaction @@ -255,11 +255,11 @@ def get_obscured_ctn(ctn, local, remote): def overall_weight(num_htlc): return 500 + 172 * num_htlc + 224 -def make_commitment(local_pubkey, remote_pubkey, +def make_commitment(local_funding_pubkey, remote_funding_pubkey, remotepubkey, payment_pubkey, remote_payment_pubkey, revocation_pubkey, delayed_pubkey, funding_txid, funding_pos, funding_satoshis, - to_local_msat, to_remote_msat, local_feerate): - pubkeys = sorted([bh2u(local_pubkey), bh2u(remote_pubkey)]) + to_local_msat, to_remote_msat, local_feerate, local_delay): + pubkeys = sorted([bh2u(local_funding_pubkey), bh2u(remote_funding_pubkey)]) obs = get_obscured_ctn(0, payment_pubkey, remote_payment_pubkey) locktime = (0x20 << 24) + (obs & 0xffffff) sequence = (0x80 << 24) + (obs >> 24) @@ -277,11 +277,12 @@ def make_commitment(local_pubkey, remote_pubkey, 'sequence':sequence }] # commitment tx outputs - local_script = bytes([opcodes.OP_IF]) + revocation_pubkey + bytes([opcodes.OP_ELSE, opcodes.OP_CSV, opcodes.OP_DROP]) + delayed_pubkey + bytes([opcodes.OP_ENDIF, opcodes.OP_CHECKSIG]) + local_script = bytes([opcodes.OP_IF]) + bfh(push_script(bh2u(revocation_pubkey))) + bytes([opcodes.OP_ELSE]) + bfh(push_script(int_to_hex(local_delay, 5))) \ + + bytes([opcodes.OP_CSV, opcodes.OP_DROP]) + bfh(push_script(bh2u(delayed_pubkey))) + bytes([opcodes.OP_ENDIF, opcodes.OP_CHECKSIG]) local_address = bitcoin.redeem_script_to_address('p2wsh', bh2u(local_script)) fee = local_feerate * overall_weight(0) // 1000 local_amount = to_local_msat // 1000 - fee - remote_address = bitcoin.pubkey_to_address('p2wpkh', bh2u(remote_pubkey)) + remote_address = bitcoin.pubkey_to_address('p2wpkh', bh2u(remotepubkey)) remote_amount = to_remote_msat // 1000 to_local = (bitcoin.TYPE_ADDRESS, local_address, local_amount) to_remote = (bitcoin.TYPE_ADDRESS, remote_address, remote_amount) @@ -289,6 +290,7 @@ def make_commitment(local_pubkey, remote_pubkey, c_outputs = [to_local, to_remote] # create commitment tx tx = Transaction.from_io(c_inputs, c_outputs, locktime=locktime, version=2) + tx.BIP_LI01_sort() return tx class Peer(PrintError): diff --git a/lib/tests/test_lnbase.py b/lib/tests/test_lnbase.py @@ -39,11 +39,11 @@ class Test_LNBase(unittest.TestCase): # local_signature = 3044022051b75c73198c6deee1a875871c3961832909acd297c6b908d59e3319e5185a46022055c419379c5051a78d00dbbce11b5b664a0c22815fbcc6fcef6b1937c3836939 #num_htlcs: 0 our_commit_tx = make_commitment( - local_funding_pubkey, remote_funding_pubkey, + local_funding_pubkey, remote_funding_pubkey, remotepubkey, local_payment_basepoint, remote_payment_basepoint, local_revocation_pubkey, local_delayedpubkey, funding_tx_id, funding_output_index, funding_amount_satoshi, - to_local_msat, to_remote_msat, local_feerate_per_kw) + to_local_msat, to_remote_msat, local_feerate_per_kw, local_delay) our_commit_tx.sign({bh2u(local_funding_pubkey): (local_funding_privkey[:-1], True)}) ref_commit_tx_str = '02000000000101bef67e4e2fb9ddeeb3461973cd4c62abb35050b1add772995b820b584a488489000000000038b02b8002c0c62d0000000000160014ccf1af2f2aabee14bb40fa3851ab2301de84311054a56a00000000002200204adb4e2f00643db396dd120d4e7dc17625f5f2c11a40d857accc862d6b7dd80e0400473044022051b75c73198c6deee1a875871c3961832909acd297c6b908d59e3319e5185a46022055c419379c5051a78d00dbbce11b5b664a0c22815fbcc6fcef6b1937c383693901483045022100f51d2e566a70ba740fc5d8c0f07b9b93d2ed741c3c0860c613173de7d39e7968022041376d520e9c0e1ad52248ddf4b22e12be8763007df977253ef45a4ca3bdb7c001475221023da092f6980e58d2c037173180e9a465476026ee50f96695963e8efe436f54eb21030e9f7b623d2ccc7c9bd44d66d5ce21ce504c0acf6385a132cec6d3c39fa711c152ae3e195220' ref_commit_tx = Transaction(ref_commit_tx_str) @@ -51,9 +51,14 @@ class Test_LNBase(unittest.TestCase): pubkeys, _x_pubkeys = our_commit_tx.get_sorted_pubkeys(our_commit_tx.inputs()[0]) index_of_pubkey = pubkeys.index(bh2u(remote_funding_pubkey)) our_commit_tx._inputs[0]["signatures"][index_of_pubkey] = remote_signature + "01" - print("our tx", str(our_commit_tx)) - print("Reference inputs", json.dumps(ref_commit_tx.inputs(), indent=2)) - print("Our inputs", json.dumps(our_commit_tx.inputs(), indent=2)) + + #print("Reference inputs", json.dumps(ref_commit_tx.inputs(), indent=2)) + #print("Our inputs", json.dumps(our_commit_tx.inputs(), indent=2)) + print("Reference outputs", ref_commit_tx.outputs()) + print("Our outputs", our_commit_tx.outputs()) + #print("Reference tx", ref_commit_tx) + #print("Our tx", our_commit_tx) + for idx, inp in enumerate(our_commit_tx.inputs()): for field in inp.keys():