-
Notifications
You must be signed in to change notification settings - Fork 231
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
Feature/nymd client integration #736
Conversation
// this behaviour should definitely be changed, we shouldn't | ||
// need to get bandwidth credential for registration |
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.
Why not? When do you think we should get it? What I had in mind was that we get it at registration/authentication and consume it during that session.
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.
My understanding is that bandwidth credential, as its name suggests, is used to access the mix network. However, during registration we are not sending any traffic through the network. Perhaps we should discuss this more with somebody else, maybe @aniampio ?
Feel free to fix the |
This pull request integrates the new validator client into the code base - mixnodes, gateways, clients and validator api are now using it.
The final remaining thing, which will be done in another pull request, is to make actual payments happen.
I'm sorry for whoever reviews it, but after the BIG MERGE ™️ this PR got rather bloated : (