Pull request for BIP75 as discussed on the mailing list
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-
techguy613 commented at 4:45 AM on March 17, 2016: contributor
-
Initial version of invoicerequest extension BIP 94c23ddffd
-
Fix mediawiki table formatting 4e3b9a5b06
-
Fix mediawiki table formatting d698201463
-
Fix mediawiki table formatting a876e47bd1
-
2c8bc2392b
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
-
Fix local anchors and links 543409c7f7
-
Remove anchor template and use spans for anchors instead 61cfbaefa7
-
Test anchor 2d9b626d6c
-
Remove anchors c91ec76f41
-
Fix last links 845f24069b
-
e71e57216b
- Fix section levels
- Add Message Interaction Details + new mimetypes
-
b2db2eba41
- 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
-
use same email for voisine as in other BIPs d14e205477
-
583a64cf3d
Merge pull request #1 from voisine/patch-3
use same email for voisine as in other BIPs
-
67470e0817
- Update motivation to include auditability
- Update InvoiceRequest to include nonce - Remove ephemeral_public_key from ReturnPaymentRequest - Update message validation and nonce usage in processes
-
- Adding James to the author list 385287b395
-
- Fix typos in abstract and "motiviation" 58754bd493
-
Updating abstract and motivation with changes made by the Netki team 5ec3d52181
-
Fix bullet in motivation 6c537eb0a7
-
Fix bullet in motivation 0c93de978e
-
added some details, fix typo a81b43b49b
-
32e82105cb
Merge pull request #2 from jmacwhyte/proofread
added some details, fix typo
-
8c78781cf0
- 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
-
Fix Implementation section spacing 88898c2f87
-
Added example use case. a715f2cdb1
-
06c3c1b488
Merge pull request #3 from jmacwhyte/proofread
Added example use case.
-
7ae2624bb3
- 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
-
62d19677c8
- 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
-
- Updated Mobile-to-Mobile image to visually include Payment and PaymentACK messages with Store & Forward server db2bf02180
-
- Fixed typos and added a bit more explanation for the Wallet Name public key retrieval example e0bd3b599a
-
961c51c3b2
editied Abstract
added "the identity of" to the first bullet of the abstract.
-
4e6ff9f149
changed justin's email back to @netkic.om
changed Justin's email back to @netki.com
-
Add indents for point 2 of the Motivation section d9bf987b99
-
e4a7582c98
Merge pull request #5 from justinwnewton/patch-3
changed justin's email back to @netkic.om
-
c7fd6ddb2e
Merge pull request #4 from justinwnewton/patch-2
editied Abstract
-
79cc9bdaa6
Update to add 3rd motivation
Adding allow for store and forward servers as a way to server payment protocol
-
f41e886f25
Request Matt to update use case 3
Asking matt to update use case 3 to include other mods made for store and forward servers.
-
101718fc6a
- Move Definitions section up
- Seperate Process sections into regular and encrypted InvoiceRequest processes and flow diagrams
-
17dfa4149e
Merge pull request #7 from justinwnewton/patch-5
Request Matt to update use case 3
-
e675fe9bdd
Merge pull request #6 from justinwnewton/patch-4
Update to add 3rd motivation
-
Merge branch 'encrypted-invoicerequest' 7a92454e54
-
Include grammar fixes from Dawn 0c3d6d15ae
-
Added EncryptedPayment and ACK, other readability and consistency fixes b01c6b7089
-
481844e49b
Merge pull request #8 from jmacwhyte/master
Encrypted payment messages, other tweaks
-
8e8c9778e9
- 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
-
52b6cde023
- Fix Signature note formatting
- Change No EncryptedPayment Required title - Fix BIP70 link format in References
-
Improved message definitions, etc. 198188b924
-
6a08dae8c4
Merge pull request #9 from jmacwhyte/proofread
Reworked message definitions, other details
-
10e6f46569
- 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
-
Made public keys required, updated steps 27bfd6165f
-
8dffb2e58a
Merge pull request #10 from jmacwhyte/proofread
Made public keys required, updated steps
-
ca0f07cfa7
- Add qualifier that nonce MUST be chosen by the encryptor.
- Also, fix ** used for bold and replace with <b></b>
-
- Update motivation for the open standards in the list for optional identity sharing 1fc921c872
-
Update bip-invoicerequest-extension.mediawiki efa3605fbe
-
Update bip-invoicerequest-extension.mediawiki df01620ebc
-
Renamed to BIP75, added extensions to BIP70 payment details c0c7d7f0a1
-
d8ec771caf
Merge pull request #12 from jmacwhyte/proofread
Renamed to BIP75, added extensions to BIP70 payment details
-
1b96cf1e78
- Update PaymentDetails index
- Added bolding to replace_by_fee
-
- 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
-
Extended BIP70 fields, added BIP number dddcb735f8
-
de42024b1a
Merge pull request #13 from jmacwhyte/proofread
Extended BIP70 fields, added BIP number
-
a8c0246295
- 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
-
481f322d44
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.
-
05740a7dbe
Format Comment
Format comment to fit style of previous comments
-
d1e1c7c000
Merge pull request #14 from jonathancross/patch-2
Formatting improvements to BIP-75
-
1f203ea5ca
Merge pull request #15 from FinalHash/master
Format Comment To Match Previous Style
-
Removed BIP70 extensions 9a5ba6688c
-
43ea7bf4a6
Merge pull request #16 from jmacwhyte/proofread
Removed BIP70 extensions
-
Merge remote-tracking branch 'upstream/master' 4c045c8801
-
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).
luke-jr added the label New BIP on Mar 17, 2016Update BIP header to pass TravisCI e8c9f67005luke-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, 2016techguy613 commented at 5:04 AM on March 17, 2016: contributorFixed the header and committed to our branch.
Merge remote-tracking branch 'origin/master' 9d86a41747techguy613 commented at 5:29 AM on March 17, 2016: contributorHey @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?
luke-jr merged this on Mar 17, 2016luke-jr closed this on Mar 17, 2016luke-jr commented at 5:47 AM on March 17, 2016: memberThe 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.
techguy613 commented at 5:15 PM on March 17, 2016: contributorThanks @luke-jr!
ContributorsLabels
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 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
More mirrored repositories can be found on mirror.b10c.me