-
Notifications
You must be signed in to change notification settings - Fork 19
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
Provide UA test vectors for the first 3 valid diversifier indices for each account #89
Conversation
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.
utACK
String-encoded UAs would be useful for testing the SDKs. Probably that change is better on another PR
CI is failing :/ |
920a0b7
to
9000e02
Compare
9000e02
to
73ced0f
Compare
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.
utACK 73ced0f
- name: Install gnome-keyring | ||
run: sudo apt-get install gnome-keyring |
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.
What is this needed for?
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.
One of the python dependencies refuses to build without it; that was the previous build failure.
This updates unified address test vectors after zcash/zcash-test-vectors#89 to check addresses across multiple diversifier indices, and also removes the superfluous binary encoding of UA strings.
No description provided.