BIP 175: Pay to Contract Protocol #586

pull omarshibli wants to merge 12 commits into bitcoin:master from commerceblock:master changing 2 files +259 −0
  1. omarshibli commented at 1:48 pm on September 6, 2017: contributor
    This proposal was submitted to the mailing list on August 14th, we got feedback and addressed the concerns, I am now requesting a BIP number.
  2. added Pay to Contract Protocol BIP draft 4ba4fc3f0a
  3. updated draft 3afd4bd57f
  4. christianrolandso commented at 3:10 pm on September 9, 2017: none
    Zzzzzzz
  5. Jatupol99 approved
  6. luke-jr added the label New BIP on Sep 16, 2017
  7. in bip-draft.mediawiki:14 in 3afd4bd57f outdated
     9+  Status: Draft
    10+  Type: Informational Track
    11+  Created: 2017-07-17
    12+</pre>
    13+
    14+&ast; 999 is a temporary BIP number.
    


    luke-jr commented at 2:52 am on September 16, 2017:
    Please don’t self-assign numbers, even temporarily.
  8. in bip-draft.mediawiki:250 in 3afd4bd57f outdated
    245+* [[bip-0044.mediawiki|BIP44 - Multi-Account Hierarchy for Deterministic Wallets]]
    246+* [[https://arxiv.org/abs/1212.3257|Homomorphic Payment Addresses and the Pay-to-Contract Protocol]]
    247+
    248+==Copyright==
    249+
    250+This document is placed in the public domain.
    


    luke-jr commented at 2:53 am on September 16, 2017:
    See BIP 2 for a list of acceptable licenses.
  9. luke-jr changes_requested
  10. luke-jr commented at 2:53 am on September 16, 2017: member
    A section is needed addressing backward compatibility.
  11. updated license 39a5c1bf7e
  12. removed temp BIP number dd26d81390
  13. added Compatibility 10104eb211
  14. omarshibli commented at 6:41 pm on September 16, 2017: contributor
    hey @luke-jr, we addressed your points, please review.
  15. luke-jr commented at 8:16 pm on September 16, 2017: member

    By compatibility, I meant more of (whatever the particular case may be; these are examples):

    • “contracts can be verified separately from the sender’s wallet software, and then a normal address is used”
    • “This is incompatible with multisig P2SH wallets”
    • “The wallet is assumed to use BIP 43 derivation”

    Note that BIP 43 (and even BIP 32 / HD wallets in general) is not universal, and your proposal’s connection to them seems unnecessary and pointless. What is the benefit of even doing anything HD-based, since the end key requires non-deterministic metadata (contracts) to find?

    Anyhow, let’s use BIP 175 for this.

  16. luke-jr renamed this:
    [New BIP] Pay to Contract Protocol
    BIP 175: Pay to Contract Protocol
    on Sep 16, 2017
  17. update Compatibility 3ff47ff642
  18. omarshibli commented at 8:19 pm on September 16, 2017: contributor
    Got it, we will work on clearing this out.
  19. updated BIP number and examples 689fcdb878
  20. updated alias 522a0e88f3
  21. updated compatibility section dd18e39624
  22. omarshibli commented at 8:26 pm on September 18, 2017: contributor

    Thanks for your feedback @luke-jr , you are raising a valid point regarding the backward compatibility with BIP32 and other related BIPs, we have updated the BIP accordingly. Regarding your question, I think we need to work better on the motivation section here, but I still see benefit here in using BIP32 derivation mechanism or something similar to derive non-enumerable set as well. I think the main benefit is that ,as you mentioned, BIP32 is not universal in a sense that it only covers use cases where the derivation paths set is an enumerable, yet in our use case we need to derive wallets from a single seed and non-enumerable set.

    Much appreciated.

  23. luke-jr commented at 10:27 pm on September 18, 2017: member
    When you’re ready, don’t forget to rename the file and add to README
  24. updated README.md, renamed file 2df903a4a1
  25. fixed number cacc29a91c
  26. omit Track 5d673da3cb
  27. in bip-0175.mediawiki:10 in cacc29a91c outdated
     5+  Author: Omar Shibli <omar@commerceblock.com>
     6+          Nicholas Gregory <nicholas@commerceblock.com>
     7+  Comments-Summary: No comments yet.
     8+  Comments-URI: https://github.com/bitcoin/bips/wiki/Comments:BIP-0175
     9+  Status: Draft
    10+  Type: Informational Track
    


    luke-jr commented at 10:54 pm on September 18, 2017:
    drop “Track”

    omarshibli commented at 10:55 pm on September 18, 2017:
    done
  28. luke-jr merged this on Sep 18, 2017
  29. luke-jr closed this on Sep 18, 2017


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bips. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-23 09:10 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me