BIP 75: Out of Band Address Exchange using Payment Protocol Encryption #357

pull techguy613 wants to merge 71 commits into bitcoin:master from techguy613:master changing 10 files +563 −1
  1. techguy613 commented at 4:45 AM on March 17, 2016: contributor

    Pull request for BIP75 as discussed on the mailing list

  2. Initial version of invoicerequest extension BIP 94c23ddffd
  3. Fix mediawiki table formatting 4e3b9a5b06
  4. Fix mediawiki table formatting d698201463
  5. Fix mediawiki table formatting a876e47bd1
  6. Updated BIP based on colleague feedback
    - Removed Requestor/Responder definitions
    - Seperated ECDH secret point generation and AES-256 (CBC Mode) setup from individual steps (listed twice) and created it's own section
    - Added InvoiceRequest Validation section
    2c8bc2392b
  7. Fix local anchors and links 543409c7f7
  8. Remove anchor template and use spans for anchors instead 61cfbaefa7
  9. Test anchor 2d9b626d6c
  10. Remove anchors c91ec76f41
  11. Fix last links 845f24069b
  12. - Fix section levels
    - Add Message Interaction Details + new mimetypes
    e71e57216b
  13. - Add flow overview image
    - Make Abstract more readable
    - Update Sender definition and acronym descriptions
    - Added comments to ReturnPaymentRequest definition
    - Bold ECDH and AES Setup notes and added "(see below)" for reference
    b2db2eba41
  14. use same email for voisine as in other BIPs d14e205477
  15. Merge pull request #1 from voisine/patch-3
    use same email for voisine as in other BIPs
    583a64cf3d
  16. - Update motivation to include auditability
    - Update InvoiceRequest to include nonce
    - Remove ephemeral_public_key from ReturnPaymentRequest
    - Update message validation and nonce usage in processes
    67470e0817
  17. - Adding James to the author list 385287b395
  18. - Fix typos in abstract and "motiviation" 58754bd493
  19. Updating abstract and motivation with changes made by the Netki team 5ec3d52181
  20. Fix bullet in motivation 6c537eb0a7
  21. Fix bullet in motivation 0c93de978e
  22. added some details, fix typo a81b43b49b
  23. Merge pull request #2 from jmacwhyte/proofread
    added some details, fix typo
    32e82105cb
  24. - Remove Acronyms section
    - Update InvoiceRequest notification_url definition to use SHOULD instead of MAY
    - Capitalize MUST, SHOULD, etc.
    - Update InvoiceRequest Message Creation steps to specifically define behavior for empty amount or amount out of bounds
    - Add implementation section with references to Addressimo reference Store & Forward server and a client implementation in functest_ir.py
    - Add flow diagrams for BIP70 extension and moble-to-mobile example with store and forward service
    8c78781cf0
  25. Fix Implementation section spacing 88898c2f87
  26. Added example use case. a715f2cdb1
  27. Merge pull request #3 from jmacwhyte/proofread
    Added example use case.
    06c3c1b488
  28. - Update e-mail addresses for Justin and Matt
    - Add 2 new use cases and add Wallet Name to the Address Book section of optional ways to add entries to an address book
    7ae2624bb3
  29. - Add encrypted invoicerequest requirement
    - Change ReturnPaymentRequest message name to EncryptedPaymentRequest
    - Add Payment Request (with Store & Forward server) use-case documentation
    - Add initial public key retrieval ideas
    62d19677c8
  30. - Updated Mobile-to-Mobile image to visually include Payment and PaymentACK messages with Store & Forward server db2bf02180
  31. - Fixed typos and added a bit more explanation for the Wallet Name public key retrieval example e0bd3b599a
  32. editied Abstract
    added "the identity of" to the first bullet of the abstract.
    961c51c3b2
  33. changed justin's email back to @netkic.om
    changed Justin's email back to @netki.com
    4e6ff9f149
  34. Add indents for point 2 of the Motivation section d9bf987b99
  35. Merge pull request #5 from justinwnewton/patch-3
    changed justin's email back to @netkic.om
    e4a7582c98
  36. Merge pull request #4 from justinwnewton/patch-2
    editied Abstract
    c7fd6ddb2e
  37. Update to add 3rd motivation
    Adding allow for store and forward servers as a way to server payment protocol
    79cc9bdaa6
  38. Request Matt to update use case 3
    Asking matt to update use case 3 to include other mods made for store and forward servers.
    f41e886f25
  39. - Move Definitions section up
    - Seperate Process sections into regular and encrypted InvoiceRequest processes and flow diagrams
    101718fc6a
  40. Merge pull request #7 from justinwnewton/patch-5
    Request Matt to update use case 3
    17dfa4149e
  41. Merge pull request #6 from justinwnewton/patch-4
    Update to add 3rd motivation
    e675fe9bdd
  42. Merge branch 'encrypted-invoicerequest' 7a92454e54
  43. Include grammar fixes from Dawn 0c3d6d15ae
  44. Added EncryptedPayment and ACK, other readability and consistency fixes b01c6b7089
  45. Merge pull request #8 from jmacwhyte/master
    Encrypted payment messages, other tweaks
    481844e49b
  46. - Add memo to InvoiceRequest message
    - Add ephemeral_public_key and requires_payment_message fields to EncryptedPaymentRequest + updated descriptions
    - Update EncryptedPayment and EncryptedPaymentACK message descriptions to use ECDH-derived key for signature instead of each side's public key
    - Slim down message content-types
    - Add EncryptedPayment and EncryptedPaymentACK creation detail steps
    - Add updated paymentrequest.proto to bip-ir/ directory
    - Add additional flow diagrams for various mobile-to-mobile / Store & Forward scenarios
    8e8c9778e9
  47. - Fix Signature note formatting
    - Change No EncryptedPayment Required title
    - Fix BIP70 link format in References
    52b6cde023
  48. Improved message definitions, etc. 198188b924
  49. Merge pull request #9 from jmacwhyte/proofread
    Reworked message definitions, other details
    6a08dae8c4
  50. - Make message public key sharing mandatory for messages that are encrypted and where both keys are known. For EncryptedInvoiceRequest, only the sender_public_key is required
    - Add nonce to EncryptedPaymentRequest, EncryptedPayment and EncryptedPaymentACK
    - Update ECDH instruction to allow for the current message instead of an InvoiceRequest to contain the nonce
    - Updated paymentrequest.proto with BIP definition changes
    10e6f46569
  51. Made public keys required, updated steps 27bfd6165f
  52. Merge pull request #10 from jmacwhyte/proofread
    Made public keys required, updated steps
    8dffb2e58a
  53. - Add qualifier that nonce MUST be chosen by the encryptor.
    - Also, fix ** used for bold and replace with <b></b>
    ca0f07cfa7
  54. - Update motivation for the open standards in the list for optional identity sharing 1fc921c872
  55. Update bip-invoicerequest-extension.mediawiki efa3605fbe
  56. Update bip-invoicerequest-extension.mediawiki df01620ebc
  57. Renamed to BIP75, added extensions to BIP70 payment details c0c7d7f0a1
  58. Merge pull request #12 from jmacwhyte/proofread
    Renamed to BIP75, added extensions to BIP70 payment details
    d8ec771caf
  59. - Update PaymentDetails index
    - Added bolding to replace_by_fee
    1b96cf1e78
  60. - Add subtract_fee and replace_by_fee flags to PaymentDetails. replace_by_fee is commented out as it's only available in version 2 of the message f753dd7372
  61. Extended BIP70 fields, added BIP number dddcb735f8
  62. Merge pull request #13 from jmacwhyte/proofread
    Extended BIP70 fields, added BIP number
    de42024b1a
  63. - Update optional flags to PaymentDetails definition in paymentrequest.proto
    - Add DER encoding requirement for EC public keys and ECC signatures
    - Add SHA-256 hashing requirement for ECC signatures
    - Add FIPS 180-4 SHS link
    a8c0246295
  64. Formatting improvements to BIP-75
     * Fixing a few extra closing `b` tags and converting others to wiki bold syntax.
     * Linking "see below" and "see above" items to the actual section of the BIP.
     * Consistent capitalization of "Bitcoin".
     * "requester" => "requester* (more common outside of legal writing)
     * "concious" => "conscious"
     * "Foward" => "Forward"
     * "Satoshis" => "satoshis" (as unit of bitcoin, not the name of creator)
     * Removing unnecessary </img> which can actually cause problems.
     * Adding required `alt` attribute to img tags.
     * Fix wrapping of long lines (some were wrapped at 112 chars) - No effect on final rendering users see.
    481f322d44
  65. Format Comment
    Format comment to fit style of previous comments
    05740a7dbe
  66. Merge pull request #14 from jonathancross/patch-2
    Formatting improvements to BIP-75
    d1e1c7c000
  67. Merge pull request #15 from FinalHash/master
    Format Comment To Match Previous Style
    1f203ea5ca
  68. Removed BIP70 extensions 9a5ba6688c
  69. Merge pull request #16 from jmacwhyte/proofread
    Removed BIP70 extensions
    43ea7bf4a6
  70. Merge remote-tracking branch 'upstream/master' 4c045c8801
  71. in bip-0075.mediawiki:None in 43ea7bf4a6 outdated
       5 | +           Matt David <mgd@mgddev.com>
       6 | +           Aaron Voisine <voisine@gmail.com>
       7 | +           James MacWhyte <macwhyte@gmail.com>
       8 | +  Status:  Draft
       9 | +  Type:    Standards Track
      10 | +  Created: 2015-11-20
    


    luke-jr commented at 5:03 AM on March 17, 2016:

    Please don't add spaces to align the headers (leave the authors list as-is).

  72. luke-jr added the label New BIP on Mar 17, 2016
  73. Update BIP header to pass TravisCI e8c9f67005
  74. luke-jr renamed this:
    Pull request for BIP75 as discussed on the mailing list
    BIP 75: Out of Band Address Exchange using Payment Protocol Encryption
    on Mar 17, 2016
  75. techguy613 commented at 5:04 AM on March 17, 2016: contributor

    Fixed the header and committed to our branch.

  76. Merge remote-tracking branch 'origin/master' 9d86a41747
  77. techguy613 commented at 5:29 AM on March 17, 2016: contributor

    Hey @luke-jr, I'm not sure what's causing this git issue exactly. Per http://stackoverflow.com/questions/13480346/git-commits-left-behind-when-switching-back-to-a-branch-from-detached-head, this is a bug that was fixed in Git v1.7.11.5 and per Travis CI, their VM are using Git 1.8. Have you seen this before?

  78. luke-jr merged this on Mar 17, 2016
  79. luke-jr closed this on Mar 17, 2016

  80. luke-jr commented at 5:47 AM on March 17, 2016: member

    The git message isn't an issue, it's normal. The issue was that you had BIP 75 after BIP 8x in the README, and didn't use the comma-only formatting for the author list there. I fixed those for you and merged it.

  81. jmacwhyte commented at 7:22 AM on March 17, 2016: contributor

    Thanks @luke-jr!

  82. techguy613 commented at 5:15 PM on March 17, 2016: contributor

    Thanks @luke-jr!


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: 2026-04-14 15:10 UTC

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