-
Notifications
You must be signed in to change notification settings - Fork 55
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
removed required prover_did #57
removed required prover_did #57
Conversation
Signed-off-by: blu3beri <blu3beri@proton.me>
anoncreds/src/data_types/mod.rs
Outdated
@@ -3,7 +3,7 @@ mod utils { | |||
pub use indy_utils::{qualifiable, Qualifiable}; | |||
} | |||
|
|||
pub use indy_utils::did; | |||
//pub use indy_utils::did; |
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.
Commented
anoncreds/src/services/prover.rs
Outdated
@@ -46,6 +48,10 @@ pub fn create_credential_request( | |||
credential_offer | |||
); | |||
|
|||
let rand_str = String::from_utf8(thread_rng().sample_iter(&Alphanumeric).take(22).collect()) |
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 value prover_did should become optional in the whole credential request. If not present in the request, the issuer will generate it.
So if no prover_did is present we don't include it in the request
Signed-off-by: blu3beri <blu3beri@proton.me>
@TimoGlastra I think I fixed the issue, it was a bit unclear to me but this should hopefully be it. |
Signed-off-by: blu3beri <blu3beri@proton.me>
9b77cd3
to
108a800
Compare
closes #52
Implemented according to: hyperledger/anoncreds-spec#107 (comment)
Also removed the dependency on
indy_utils::DidValue
which makes #40 a whole lot easier.Signed-off-by: blu3beri blu3beri@proton.me