After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 761506 - Rename SSL to TLS
Rename SSL to TLS
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: general
3.18.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: Evolution Shell Maintainers Team
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2016-02-03 14:26 UTC by Daniel Aleksandersen
Modified: 2016-02-09 09:33 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Daniel Aleksandersen 2016-02-03 14:26:40 UTC
Now that SSL is completely deprecated (see RFC 7568), all UI strings should be change all mentions from ”SSL” to “TLS”. SSL is mentioned quite a few places, especially in account and server settings.

This bug is not about blocking or removing SSL from Evolution, but merely about relabling the technology so the product feels fresh rather than stale and referring to deprecated technology.
Comment 1 Milan Crha 2016-02-05 14:30:40 UTC
Thanks for a bug report. I agree with the "rename". I'm only wondering, when the UI has a pickup from values like:

   Encryption method:  No encryption
                       STARTTLS after connecting
                       SSL on a dedicated port

Then it might become:

   Encryption method:  No encryption
                       STARTTLS after connecting
                       TLS on a dedicated port

which could confuse regular (non-tech) users, seeing TLS listed twice. Even I would still not remove the TLS from the string completely, but maybe this way would make it better:

                       Secure connection (TLS)

What do you think? Of course, some places would be simple replace, but this one seems like the most common place with it visible.
Comment 2 Daniel Aleksandersen 2016-02-05 19:57:22 UTC
If you want it to be user friendly, you’d need a redesign. One checkbox that simply says “Allow insecure server connections.” (disabled by default – as the default should be the reverse , meaning encrypted connections are required). Try to connect using TLS on the provided port, try STARTTLS on the provided port, [optionally try STARTTLS on the standard 143 port disregarding the user provided port], and then display an error message. The error would say “Cannot make encrypted connections to this server” or “couldn’t find an IMAP server”. Having the user pick between STARTTLS and TLS is meaningless for everyone but the very few users/sysadmins who are testing one or the other, and they can probe their server in other ways.

Otherwise STARTTLS and TLS in a drop-down menu is exactly and technically correct. It’s no more complicated than STARTTLS and SSL. Both variants are basically letter soups for the uninitiated anyways.

Whichever option is chosen, it should also be synced with GOA. They have the very same drop-down menu when adding an IMAP/SMTP account.
Comment 3 Milan Crha 2016-02-08 16:53:17 UTC
(In reply to Daniel Aleksandersen from comment #2)
> Otherwise STARTTLS and TLS in a drop-down menu is exactly and technically
> correct. It’s no more complicated than STARTTLS and SSL. Both variants are
> basically letter soups for the uninitiated anyways.

Okay, then I'll simply replace SSL with TLS in the evolution/evolution-data-server code.

> Whichever option is chosen, it should also be synced with GOA. They have the
> very same drop-down menu when adding an IMAP/SMTP account.

Please open a separate bug report against GOA, the maintainer of it most likely doesn't follow bug reports of evolution/-data-server.
Comment 4 Milan Crha 2016-02-09 09:33:09 UTC
It didn't feel right to make TLS from SSL everywhere, because there is no (easy) way to know which had been used in the background, thus I somewhere changes SSL to TLS and somewhere SSL to SSL/TLS. I changed onlo developer documentation in a similar way.

Created commit_716662a in eds master (3.19.90+) [1]
Created commit_2bd0cef in evo master (3.19.90+) [2]

[1] https://git.gnome.org/browse/evolution-data-server/commit/?id=716662a
[2] https://git.gnome.org/browse/evolution/commit/?id=2bd0cef