Skip to content
This repository has been archived by the owner on Feb 13, 2024. It is now read-only.

Proof Template: Allow multiple restrictions #638

Merged
merged 12 commits into from
Sep 30, 2021

Conversation

etschelp
Copy link
Contributor

@etschelp etschelp commented Sep 27, 2021

  • addresses Backend: Restriction Objects #637 it is now possible to add one or many restrictions to a proof request
  • error events are now handled on both sides (holder, verifier), so we now see if the proof request was illegal. This can happen for example when there are two credentials in the wallet one with name=10, and one with name=MyName and we send a proof request requesting name to be < 11. In this case aca-py logs an error and ends the proof request immediately.
  • schema version and schema name are not pre-filled any more
  • the backend truncates trusted and schema issuer dids if they are fully qualified, as aca-py only allows the last segment of the did at the moment for proof requests v1
  • fixed bug in the credential component

Signed-off-by: Philipp Etschel <philipp.etschel@ch.bosch.com>
Signed-off-by: Philipp Etschel <philipp.etschel@ch.bosch.com>
@etschelp etschelp changed the title Feature/637 Proof Template: Allow multiple restrictions Sep 27, 2021
@etschelp etschelp marked this pull request as draft September 27, 2021 15:18
Signed-off-by: Philipp Etschel <philipp.etschel@ch.bosch.com>
Signed-off-by: Philipp Etschel <philipp.etschel@ch.bosch.com>
Signed-off-by: Philipp Etschel <philipp.etschel@ch.bosch.com>
…it will switch to abandoned immediately.

Signed-off-by: Philipp Etschel <philipp.etschel@ch.bosch.com>
…redundant

Signed-off-by: Philipp Etschel <philipp.etschel@ch.bosch.com>
@etschelp etschelp marked this pull request as ready for review September 28, 2021 12:34
Signed-off-by: Philipp Etschel <philipp.etschel@ch.bosch.com>
Signed-off-by: Philipp Etschel <philipp.etschel@ch.bosch.com>
Signed-off-by: Philipp Etschel <philipp.etschel@ch.bosch.com>
@etschelp etschelp merged commit 8a73c6c into hyperledger-labs:main Sep 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant