-
Notifications
You must be signed in to change notification settings - Fork 646
Decode minikeys (like Casascius coin ones) #141
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
Open
juestr
wants to merge
7
commits into
petertodd:master
Choose a base branch
from
juestr:minikey
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 3 commits
Commits
Show all changes
7 commits
Select commit
Hold shift + click to select a range
266a9fc
Add minikey decoding.
juestr bafd3dc
Add a valid 22 char minikey to test vector.
juestr 88b5866
Remove unneeded import of sha256.
juestr 6fd0e66
Revert base58 files.
juestr e9713ad
Add minikey module, using CBitcoinSecret for private keys.
juestr db7b8f9
Fix minikey cli tool in examples.
juestr 0e86ba2
Add bitcoin.minikey line to README.
juestr File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,36 @@ | ||
| #!/usr/bin/env python3 | ||
| # | ||
| # Copyright (C) 2013-2015 The python-bitcoinlib developers | ||
| # | ||
| # This file is part of python-bitcoinlib. | ||
| # | ||
| # It is subject to the license terms in the LICENSE file found in the top-level | ||
| # directory of this distribution. | ||
| # | ||
| # No part of python-bitcoinlib, including this file, may be copied, modified, | ||
| # propagated, or distributed except according to the terms contained in the | ||
| # LICENSE file. | ||
|
|
||
| from __future__ import absolute_import, division, print_function, unicode_literals | ||
|
|
||
| from bitcoin import base58 | ||
|
|
||
| def parser(): | ||
| import argparse | ||
| parser = argparse.ArgumentParser( | ||
| description='Decode a minikey to base58 format.', | ||
| epilog='Security warning: arguments may be visible to other users on the same host.') | ||
| parser.add_argument( | ||
| 'minikey', | ||
| help='the minikey') | ||
| return parser | ||
|
|
||
| if __name__ == '__main__': | ||
| args = parser().parse_args() | ||
| try: | ||
| base58_key = base58.decode_minikey(args.minikey) | ||
| except Exception as error: | ||
| print('%s: %s' % (error.__class__.__name__, str(error))) | ||
| exit(1) | ||
| else: | ||
| print(base58_key) |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggest using
str("length is {} yo").format(length)instead of using the%operator.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you please explain why?