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

proposed text from errata 6648 #151

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

proposed text from errata 6648 #151

wants to merge 6 commits into from

Conversation

mcr
Copy link
Member

@mcr mcr commented Feb 20, 2024

No description provided.

Comment on lines 1745 to 1748
When TLS 1.3 is used the use of Server Name Indicator (SNI,
<xref target="RFC6066" />) is not required, per <xref
target="RFC8446" /> section 9.2, this
specification is an application profile specification.
Copy link
Member Author

@mcr mcr Feb 20, 2024

Choose a reason for hiding this comment

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

Suggested change
When TLS 1.3 is used the use of Server Name Indicator (SNI,
<xref target="RFC6066" />) is not required, per <xref
target="RFC8446" /> section 9.2, this
specification is an application profile specification.
While the use of a Server Name Indication (SNI) is not required per this (<xref target="RFC8446" />, Section 9.2) application profile specification, pledges MAY include one.
Registrars MUST ignore any SNI that they see.

(This aligns with BRSKI-CLOUD needs)

Choose a reason for hiding this comment

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

suggesting to change to "Server Name Indication (SNI)" (i.e. use the official term) and to use lowercase on "application profile specification" and for "pledges". (Conforms to RFC 8995 caps use)

We can also commit this suggestion soon to allow further wordsmithing.

Copy link
Member Author

Choose a reason for hiding this comment

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

( I can't commit my own suggestion, it seems)

mcr and others added 5 commits February 21, 2024 12:16
Co-authored-by: Esko Dijk <[email protected]>
Co-authored-by: Esko Dijk <[email protected]>
Co-authored-by: Esko Dijk <[email protected]>
Co-authored-by: Esko Dijk <[email protected]>
Co-authored-by: Esko Dijk <[email protected]>
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