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

Server now generates addresses #14

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

SoulByte
Copy link
Contributor

The BlooCoin server now generates the address for registering clients.

The BlooCoin server now generates the address for registering clients.
@sysr-q
Copy link
Contributor

sysr-q commented May 23, 2013

For one, Jesus H. Christ, use uuid.uuid4() for your random stuff, that random.randint(0, 100000000000) is killing the pythonista in me.
My question beside that is: what is gained from having the server generate addresses? It's going to get rid of vanity addresses is all, and correct me if I'm wrong (which I probably am) but couldn't you generated a BitCoin vanity address in the same way you can generate a vanity Tor onion: bruteforcing those motherfuckers.
This just seems irritating.

@ajvpot
Copy link

ajvpot commented May 27, 2013

This would likely encourage brute forcing to generate vanity addresses.

@sysr-q
Copy link
Contributor

sysr-q commented May 27, 2013

Agreed. This is more hassle than its worth, imo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants