1122666 - Default IRC to using SSL

Status: RESOLVED FIXED

(Chat Core :: IRC, enhancement)

Description

[Patrick Cloke [:clokep]]

Attachment: irc-ssl.diff

Since bug 954724 seems to be fixed now we can default to SSL for IRC.

Comment 1

[aleth [:aleth]]

Comment on attachment 8550419 [details] [diff] [review]
irc-ssl.diff

Review of attachment 8550419 [details] [diff] [review]:
-----------------------------------------------------------------

Yes please!

Comment 2

[Patrick Cloke [:clokep]]

A little justification.

2:10:53 PM - flo-retina: is 6697 a standard port for SSL IRC?
2:11:30 PM - flo-retina: I thought last time we discussed this we wanted to make that change per-network rather than globally
2:11:46 PM - aleth: 6697 is standard
2:11:57 PM - flo-retina: are ALL the large IRC networks supporting SSL?
2:12:04 PM - flo-retina: if not, we are going to break existing accounts
2:12:48 PM - aleth: flo-retina: There used to be a freenode SSL bug which was blocking making this change.
2:15:02 PM - aleth: flo-retina: I guess we'll soon get feedback if there is a major IRC network we've overlooked (with no SSL or a different port)
2:16:25 PM - aleth: "All IRC networks, except one, in the global top twenty which offer IRC via SSL/TLS are listening on port 6697 and many smaller networks do, as well" was written in 2011

Thanks for the review.

Comment 3

[aleth [:aleth]]

https://hg.mozilla.org/comm-central/rev/f28ff368b740