-
Notifications
You must be signed in to change notification settings - Fork 14
Mention HTTPS bindings as mandatory #227
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: main
Are you sure you want to change the base?
Conversation
|
I'm generally open to this amendment. The topic hasn't been discussed in the committer's round yet so I think a PR is a little early at this stage. Up until now, relying specifications had to specifically reference the binding to lift it to normativity, see this example_
The committers have to discuss the effect on the spec's releases as it's definitively a change with a normative effect. |
|
Hi, |
ssteinbuss
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.
Requirements cannot be stated in the introduction. The addition does not add value to the text. It is misleading and requires more explanations, what goes beyond the scope of the introduction.
Proposal is to reject this contribution.
|
I understand the feedback and the text can be moved to the appropriate section if one is given. The proposal is to clarify that, in this section:
This makes |
|
As far as I can see:
For the versioning endpoint, an HTTPS response is mandatory. However, different versions can be offered. At the moment, only the HTTPS binding exists, but the DSP shall be open to other bindings. This would include the need for a specification of the binding and an extension of the TCK. But the idea is not to make HTTPs mandatory. Thus, it should not be stated like this in the document. |
For the purpose of forward-compatibility (and opposite to my initial reaction) it might be smarter to leave it up to each dataspace which binding to declare mandatory. Otherwise, participants in a future dataspace using DSP over, say, AMQP would be required to expose two sets of endpoints for the same interactions. |
What this PR changes/adds
Clarify that HTTPS bindings are mandatory
(and editorial changes to have one sentence per line, simplifying the future PRs and review)
Linked Issue(s)
International-Data-Spaces-Association/identity-in-data-spaces#11 (comment)