-
Notifications
You must be signed in to change notification settings - Fork 1
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
admin api for generic tails file upload #125
admin api for generic tails file upload #125
Conversation
Signed-off-by: Adam Burdett <burdettadam@gmail.com>
b72608c
to
57d1e4b
Compare
Signed-off-by: Adam Burdett <burdettadam@gmail.com>
Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
aries_cloudagent/anoncreds/routes.py
Outdated
try: | ||
issuer = AnonCredsIssuer(profile) | ||
rev_reg_def_result = ( | ||
await anoncreds_registry.get_revocation_registry_definition( |
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.
The check we're performing here should be making sure that we have the rev reg def locally so we can upload the tails file (which should also be present locally). The get_revocation_registry_definition
method on the anoncreds registry retrieves a rev reg def from a remote registry though. So what I think we want is actually something more like get_created_revocation_registry_definitions
, similar to the method we have for cred defs.
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.
Or rather, I think I was thinking of match_created_credential_definitions
Signed-off-by: Daniel Bluhm <dbluhm@pm.me>
7d2e573
into
feature/anoncreds-revocation-again
No description provided.