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

buflist: Do not hardcode colors in tls_version #1684

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

KaiserBarbarossa
Copy link

Do not hardcode the colors in buflist.format.tls_version

@flashcode flashcode added the feature New feature request label Aug 11, 2021
@ksu-0
Copy link

ksu-0 commented Oct 1, 2021

This incorrectly marks TLS 1.2 as deprecated. At the moment, deprecated versions are TLS 1.1 and earlier, and DTLS 1.0.

@KaiserBarbarossa
Copy link
Author

This incorrectly marks TLS 1.2 as deprecated.

That’s what is done at the moment too and which was all introduced in #1622 (Suggested by this comment). As you can see here, too:

case GNUTLS_TLS1_2:
color = IRC_COLOR_ITEM_TLS_VERSION_DEPRECATED;
break;

@flashcode
Copy link
Member

@ksu-0: where can we check which versions of TLS are deprecated or not (today)?

@ksu-0
Copy link

ksu-0 commented Oct 2, 2021

The authoritative source for SSL 3.0 and newer, and for DTLS is the status of RFC that defines it. Moreover, each deprecation (including one for the never standardized SSL 2.0) was announced by RFC.

version RFC status deprecation RFC
TLS 1.3 8446 proposed standard
TLS 1.2 5246 proposed standard
TLS 1.1 4346 historic (changed from proposed standard) 8996
TLS 1.0 2246 historic (changed from proposed standard) 8996
SSL 3.0 6101 historic 7568
SSL 2.0 expired draft 6176
DTLS 1.2 6347 proposed standard
DTLS 1.0 4347 historic (changed from proposed standard) 8996

@talisein
Copy link
Contributor

talisein commented Feb 7, 2022

My choice to use the word "deprecated" in the color option was imprecise. That color option is meant to draw attention to servers that are not using the latest TLS version but are still secure. The versions currently officially deprecated are denoted by the "insecure" color option.

I think this PR is good. Adjusting the color option name, if needed, can be done separately (this deprecated word is used elsewhere throughout the code for TLS 1.2).

@KaiserBarbarossa
Copy link
Author

What about renaming "deprecated" to "elder"?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants