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

modifications to main derivation sections #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

markblundeberg
Copy link
Contributor

This is a consolidation and rewrite of the main sections on prefix / derivation.

  • Put emphasis on how recipients must behave, as this is where most complexity exists.
    Senders in turn need to respect this, and it is their responsibility to not generate
    funds-losing transactions.
  • Introduce P2SH-multisig key sorting on outputs, to have the generated multisigs appear typical.
  • Change how the outpoint is included in the secret derivation (bytestring concatenation instead of
    integer addition, which was incompletely described before).
  • Clarify input-prefix computation in its own section; clarify target-prefix extraction from payment code.
  • Carefully specify how wallets may classify & analyze input types by scanning the scriptSig alone.
    (this is what EC will do, for instance).

This is a consolidation and rewrite of the main sections on prefix / derivation.

- Put emphasis on how recipients must behave, as this is where most complexity exists.
  Senders in turn need to respect this, and it is their responsibility to not generate
  funds-losing transactions.
- Introduce P2SH-multisig key sorting on outputs, to have the generated multisigs appear typical.
- Change how the outpoint is included in the secret derivation (bytestring concatenation instead of
  integer addition, which was incompletely described before).
- Clarify input-prefix computation in its own section; clarify target-prefix extraction from payment code.
- Carefully specify how wallets may classify & analyze input types by scanning the scriptSig alone.
  (this is what EC will do, for instance)
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.

1 participant