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

IANA actions to establish "co" ALPN #3

Merged
merged 2 commits into from
Feb 29, 2024
Merged

IANA actions to establish "co" ALPN #3

merged 2 commits into from
Feb 29, 2024

Conversation

chrysn
Copy link
Collaborator

@chrysn chrysn commented Feb 28, 2024

This picks up https://mailarchive.ietf.org/arch/msg/tls-reg-review/meMm8LYSxvn4BGIkikScM3BblOw/ -- once that is out in a draft, we'll get the ALPN.

We don't need to worry about document normativity here, the registry is expert review only -- they just want some place that has text.

And spare us a round trip: Yes, this is a problem statement document. If you insist on problem-statement-ish text, that may be

The problem is trival for the DTLS case, as illustrated in section IANA considerations – those lines and a mail to the list was all it took to just register it. For the non-transport layer security layers, we're writing all the rest of that text.

(Granted, the document might warrant a less flippant tone, but I hope it suffices to convince you that it does indeed fit in this document.)

This could go into literally any other related document as well, but this one is most suitable because for the time being, this is the only document that does something with it, even if it's just illustrative examples. If it turns out that our solutions document also covers part of the TLS case, there is nothing to stop us from moving the comment as the document is advanced, and asking the experts to update the reference.

Comment on lines 290 to 291
The TLS registration review team was asked to enter the following into the registry called
TLS Application-Layer Protocol Negotiation (ALPN) Protocol IDs:
Copy link
Contributor

Choose a reason for hiding this comment

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

Everytime I read this, I stumble over it... this usually means something is not right ;-). Also, I think it helps finding the entry bye also naming the registry above the sub-registry. How about

Suggested change
The TLS registration review team was asked to enter the following into the registry called
TLS Application-Layer Protocol Negotiation (ALPN) Protocol IDs:
The following entry was added to the “TLS Application-Layer Protocol Negotiation (ALPN) Protocol IDs” sub-registry of the ”Transport Layer Security (TLS) Extensions“ in consultation with the TLS registration review team:

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

"was added" will only be correct in the next revision; "is requested" is the usual terminology for while a document will request something at publication, and it's changed once IANA has done it (shortly before publication) to "has been requested" -- so in this situation "team was asked" sounds right to me. It's also not a sub-registry (cf. faq).

The rest of the changes I'm working into an update; may not even need the review team mention in there because that is implied by how the registration works, and more importantly not helpful to the reader.

@chrysn
Copy link
Collaborator Author

chrysn commented Feb 29, 2024

If we'd manage to publish the -00 ahead of the cut-off (say, tomorrow?), it's well possible that we can already do a -01 with "has been added to".

@miri64
Copy link
Contributor

miri64 commented Feb 29, 2024

If we'd manage to publish the -00 ahead of the cut-off (say, tomorrow?), it's well possible that we can already do a -01 with "has been added to".

What do you say @waehlisch, @tcschmidt? We can also iterate to -01 until monday which would include your feedback.

@miri64 miri64 merged commit b71c0b4 into main Feb 29, 2024
2 checks passed
@chrysn chrysn deleted the iana-for-alpn branch February 29, 2024 17:57
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.

2 participants