Skip to content
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

Make cTLS extensible #45

Merged
merged 2 commits into from
Mar 7, 2022
Merged

Make cTLS extensible #45

merged 2 commits into from
Mar 7, 2022

Conversation

bemasc
Copy link
Collaborator

@bemasc bemasc commented Feb 23, 2022

Fixes #39

@hannestschofenig
Copy link
Contributor

This PR looks good to me.

: contains keys that are not required to be understood by the client.
The server MUST be able to accept connections that do or do not use any of
these keys. A key MUST NOT appear in both the main template and the optional
section.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bemasc.

I like the general idea here, but it's not clear to me how this is going to work. Like how do I infer whether the key is in use or not? Can you give an example?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, this can only be used for keys that the server can disambiguate. I'm not necessarily claiming that any such keys have been defined, but I think we might as well agree ahead of time on where to put them, for forwards-compatibility.

My main point here is that all the other keys are not optional, and if you don't recognize one then you can't use the template.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, that's fair. Could you add some text to the effect about disambiguate and I'll merge thjs?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@ekr ekr merged commit 9e406d6 into tlswg:master Mar 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Request: Clarify extensibility rules
3 participants