-
Notifications
You must be signed in to change notification settings - Fork 49
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
Feat: a new API for extended disclosure proofs #245
Feat: a new API for extended disclosure proofs #245
Conversation
… attributes, proofs, publickeys and other meta data
…d plain gabi.publickey (ECDSA does not have a unmarshal strategy)
…is something every request will probably do in the future
…e protocol in disclosure result (required to validate user master secret zkp)
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.
Hi Sten,
Thanks. Apart from some minor things which I have put in some comments, the code itself looks fine.
Mostly, as some of my comments indicate, I am unsure about whether the point of this PR is to create a datastructure that can be verified independently, i.e., without trusting the IRMA server itself. I think that is very difficult to achieve, and indeed if that is the purpose of this PR then I believe it does not fully achieve it. If not, then what exactly would the verifier want with all this extra information?
|
|
Merged latest master commits into this feature. @ivard Is there any progress on what you guys want to do with this PR? It has been open for over a year now. |
After consultation with @ivard, I decided to close the pull request due to other priorities at my company. We will be no longer maintaining two separated branches. |
Following a discussion at the last IRMA meetup on disclosure proofs in the API with @ivard, I created a new endpoint that exposes more information about the disclosure (compatible both for signature based requests and normal disclosure based requests).
This new endpoint provides:
The new endpoint
/session/{requestorToken}/result-extended
results in aResultExtended
that provides the following properties:request
: A stateless way to obtain request parameterstype
: Either 'signing' or 'disclosing'status
: A status flag to see the statusproofStatus
: A status flag to see if the includes a valid proofnonce
: The real nonce used in the proofcredentials
: An array of credentialsidentifier
: A CredentialTypeIdentifierissuedAt
: A unix timestamp that shows when the credential was issuedexpiresAt
: A unix-timestamp that shows when the credential will expireissuer
: An object containing issuer informationidentifier
: A IssuerIdentifierscheme
: An object containing scheme informationidentifier
: A SchemeManagerIdentifierdistributedKey
: A boolean indicating the scheme uses a keyshare serverattributes
: An array of attributesidentifier
: An AttributeTypeIdentifiervalue
: The value of the attribute (or null)status
: The status of the attribute valueproof
: A proof object, equal to a gabi.ProofDAn example of a successful disclosure and the corresponding JSON result from
/session/{requestorToken}/result-extended
: