Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Require access to private keys for wallets #1706

Merged
merged 1 commit into from Aug 3, 2017

Conversation

Projects
None yet
3 participants
Contributor

crwatkins commented Jul 25, 2017

This PR codifies what I believe to be our de facto criteria for the last two years. There has been a fair amount of discussion et al. with very little support for listing wallets that do not allow access to private keys. For over two years we have not listed any new wallets that do not allow access to private keys. Instead, we have listed such submissions elsewhere on the site under other categories.

Contributor

schildbach commented Jul 25, 2017

Just checking, this would essentially remove Coinapult, Xapo and Coinbase from the web wallets section? All others grant you full or shared control over your money.

Contributor

crwatkins commented Jul 25, 2017

@schildbach I left that out of this PR for now so that we could discuss the criteria and let it stand on its own, but after we all settle on any new criteria I would submit a new PR to update the listings. If the wording remains the same, the answer to your question is yes those web wallets would most likely be removed. Note that in this PR we already moved these wallets, but have not yet removed them from the wallet listings.

Note that Coinbase has a non-default wallet mode which I believe allows for access to private keys. Even though our normal criteria are applied against default operating modes, I would be willing to consider this for an exception if Coinbase or another party would be interested in proposing and supporting the listing.

@wbnns wbnns added the Under Review label Jul 28, 2017

Contributor

wbnns commented Jul 31, 2017

Unless others object, this will be merged on Wednesday, August 2nd.

@wbnns wbnns added Merge Scheduled and removed Under Review labels Jul 31, 2017

@wbnns wbnns merged commit c2ee71b into bitcoin-dot-org:master Aug 3, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment