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

Rename the "Supported Elliptic Curves" extension. #4

Open
grubba opened this issue Jan 13, 2015 · 4 comments
Open

Rename the "Supported Elliptic Curves" extension. #4

grubba opened this issue Jan 13, 2015 · 4 comments

Comments

@grubba
Copy link

grubba commented Jan 13, 2015

This extension has been renamed "Supported Groups" in the drafts of TLS 1.3 and FFDHE.

Having different names for the same extension in different concurrent RFCs should be avoided.

Section 5.1.1 and other places should be updated accordingly.

@yoavnir
Copy link
Contributor

yoavnir commented Jan 13, 2015

Hi, Henrik.

The proper place to discuss the draft in the TLS mailing list. Would you
mind posting there?

To the question, we are revving an existing document. This extension
already has a name, and it appears in multiple code bases as well as
protocol analyzers like wireshark and in documents, websites and books.

I don't think we should change its name in TLS 1.2, but we should take this
to the list.

Yoav

On Tue, Jan 13, 2015 at 1:26 PM, Henrik Grubbström (Grubba) <
[email protected]> wrote:

This extension has been renamed "Supported Groups" in the drafts of TLS
1.3 and FFDHE.

Having different names for the same extension in different concurrent RFCs
should be avoided.

Section 5.1.1 and other places should be updated accordingly.


Reply to this email directly or view it on GitHub
#4.

@grubba
Copy link
Author

grubba commented Jan 14, 2015

Ok, done so.

@Lekensteyn
Copy link

FWIW, RFC 7919 (August 2016) has renamed the IANA registry and the elliptic_curves extension too for TLS 1.0, 1.1 and 1.2.

@seanturner
Copy link
Contributor

This is noted in the draft in s5.1.1:

RFC 4492 defined 25 different curves in the NamedCurve registry (now
renamed the "Supported Groups" registry, although the enumeration
below is still named NamedCurve) for use in TLS.

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

No branches or pull requests

4 participants