Closed
Bug 1122567
Opened 9 years ago
Closed 9 years ago
Change JS-XMPP default to require encryption
Categories
(Chat Core :: XMPP, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
1.6
People
(Reporter: aleth, Assigned: aleth)
Details
Attachments
(1 file)
|
2.01 KB,
patch
|
clokep
:
review+
|
Details | Diff | Splinter Review |
No description provided.
|
Assignee | |
Comment 1•9 years ago
|
||
While looking at the surrounding code, I noticed we never use STARTTLS for gtalk due to https://dxr.mozilla.org/comm-central/source/chat/protocols/xmpp/xmpp-session.jsm#32, instead attempting SSL directly. Might be worth checking if removing the port 443 check there helps with the reconnection issue?
|
||
Comment 2•9 years ago
|
||
Comment on attachment 8550461 [details] [diff] [review] requiretls.diff Review of attachment 8550461 [details] [diff] [review]: ----------------------------------------------------------------- Are we confident most servers have SSL support? I assume facebook does but hope you tested. :-)
Attachment #8550461 -
Flags: review?(clokep) → review+
|
|
Assignee | |
Comment 3•9 years ago
|
||
SSL support is reported on https://xmpp.net/. Stats are https://xmpp.net/reports.php
Keywords: checkin-needed
|
|
Assignee | |
Comment 4•9 years ago
|
||
https://hg.mozilla.org/comm-central/rev/14d3088e8af2
|
|
Assignee | |
Updated•9 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 1.6
You need to log in
before you can comment on or make changes to this bug.
Description
•