-
Notifications
You must be signed in to change notification settings - Fork 0
RFC056: Update OpenAPI spec for RFC056 (DOC-73) #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
base: develop
Are you sure you want to change the base?
RFC056: Update OpenAPI spec for RFC056 (DOC-73) #19
Conversation
ishare_openapi_spec.yaml
Outdated
| Optional id of the Service (Capability) of the Service Provider for which | ||
| this AR is selected by the Entitled Party. | ||
| type: string | ||
| example: /service1 |
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.
this should be an identifier and. not necessarily an endpoint. To make it more clear, you could refer to the attribute in the capabilities endpoint
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.
Improved in newer version.
| the Entitled Party. | ||
| type: string | ||
| example: did:ishare:EU.NL.NTRNL-12345678 | ||
| serviceIdentifier: |
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.
service id was not agreed in parties endpoint, that will lead to lots of entries
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.
I think it is. See https://gitlab.com/ishare-foundation/cab/rfc/-/blob/f003a57e55516272998bc42353a61e29d23927c0/RFC%20Documents/RFC056/README.md option 1 "If the Entitled Party has registered a capabilities endpoint in which an AR is registered that must be used for a specific capability, then that AR must be used.".
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.
but that is not correct, it was never agreed, it was only discussed, but the implications are too big. I cannot approve it.
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.
combined solutions chosen in not very well explained. capability reference needs to be standardised and it must still be looked up in capabilities endpoint for the URL and other details. and don't forget the overhead and potential for errors.
rajiv-ishare
left a comment
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.
released
No description provided.