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

BIP 127: Simple Proof-of-Reserves Transactions #756

Merged
merged 1 commit into from Apr 4, 2019

Conversation

stevenroose
Copy link
Contributor

@stevenroose stevenroose commented Jan 29, 2019

(submitted to bitcoin-dev, awaiting approval)

@luke-jr
Copy link
Member

luke-jr commented Feb 15, 2019

This needs a backward compatibility section (probably simply saying it's n/a).

Also, @achow101 needs to ACK the modifications to BIP 174.

@achow101
Copy link
Member

You should add the new type to BIP 174's Appendix A. That should also include a link to the location in the new BIP descriibing it.

@stevenroose
Copy link
Contributor Author

@achow101 I added the field to that table. I also mention the BIP in the explanation, but it's currently set to TBD until this BIP gets a number assigned, then I can update both mentions of TBD to become a hyperlink.

@luke-jr added compatibility section.

@luke-jr luke-jr changed the title [proposal] Simple Proof-of-Reserves Transactions BIP 127: Simple Proof-of-Reserves Transactions Feb 16, 2019
@luke-jr
Copy link
Member

luke-jr commented Feb 16, 2019

Assigned BIP 127

@stevenroose
Copy link
Contributor Author

stevenroose commented Feb 17, 2019

I just realized it might be useful to have a way to deal with funds that are locked in a CLTV in the future. Are their opinions about this?

I'm thinking of two approaches:

  1. have a global "locktime" variable for the entire proof, which proves that "the prover would be able to spend C coins at time L"
  2. allow specifying a locktime for some UTXOs which allows for more fine-grained locktime limitation specification, but might make the "proof statement" more complex. Something like "the prover has ownership of C1 coins now and C2 coins a time L1, ...". For UX purposes the prover can decide to use the same locktime for all it's CLTV UTXOs even though the locktime is further away for some of the UTXOs. Just to simplify the statement.

Both techniques would be quite straightforward to implement for verification.

@stevenroose
Copy link
Contributor Author

@luke-jr I updated the files to use the assigned number. Up to you if you merge it like this and allow me to update once I figured out a good way to deal with CLTV or if you want to wait for that in order to merge this.

@luke-jr
Copy link
Member

luke-jr commented Feb 18, 2019

"the prover would be able to spend C coins at time L"

This can't be proven. The TXOs might get spent before time L.

@luke-jr
Copy link
Member

luke-jr commented Feb 18, 2019

First Comments-URI must be exactly "https://github.com/bitcoin/bips/wiki/Comments:BIP-0127" in bip-0127.mediawiki

@achow101
Copy link
Member

ACK BIP174 changes

@stevenroose
Copy link
Contributor Author

@luke-jr There is an error in the Travis that I don't understand. Seems to be related to the git history. I tried rebasing from latest master but that didn't work..

@stevenroose
Copy link
Contributor Author

@luke-jr ping :)

@luke-jr
Copy link
Member

luke-jr commented Mar 18, 2019

It's saying you need to update the README

bip-0127.mediawiki Outdated Show resolved Hide resolved
bip-0127.mediawiki Outdated Show resolved Hide resolved
@stevenroose
Copy link
Contributor Author

@luke-jr fixed the README

I don't currently have much time to work on this, but I think it's quite final apart from the CLTV issue. So can go in like this and I can amend with a fix for that if needed.

@luke-jr luke-jr merged commit e65e3fa into bitcoin:master Apr 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants