Skip to content

use p2pkh vout instead of p2pk for change when -pubkey specified#638

Merged
DeckerSU merged 1 commit intodevfrom
patch-use-p2pkh-4-change-with-pubkey
Oct 17, 2024
Merged

use p2pkh vout instead of p2pk for change when -pubkey specified#638
DeckerSU merged 1 commit intodevfrom
patch-use-p2pkh-4-change-with-pubkey

Conversation

@DeckerSU
Copy link
Copy Markdown

@DeckerSU DeckerSU commented Oct 16, 2024

As is well known, Iguana uses P2PKH UTXOs for splitting and converts them into 10,000 zat P2PK UTXOs for notarizing. However, it cannot split from P2PK UTXOs. When you run komodod with the -pubkey argument, specifying your notary pubkey, any transaction created inside the daemon will send change (if any) as a P2PK output, i.e., to the <pubkey> OP_CHECKSIG script. As a result, Iguana cannot split such UTXOs. This change modifies that behavior, making the change a standard P2PKH output if a pubkey is specified.

this will help Iguana identify the change UTXOs and use them for splitting.
Copy link
Copy Markdown

@smk762 smk762 left a comment

Choose a reason for hiding this comment

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

Nice, this should help stop running out of utxos while a balance exists.

@DeckerSU
Copy link
Copy Markdown
Author

Nice, this should help stop running out of utxos while a balance exists.

Yes, except the cases when all remaining balance it's generated coins.

@DeckerSU DeckerSU merged commit 2651eaf into dev Oct 17, 2024
TheComputerGenie pushed a commit to ComputerGenieCo/komodo that referenced this pull request Sep 12, 2025
…ECBTC#638)

this will help Iguana identify the change UTXOs and use them for splitting.
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