Closed Bug 1122567 Opened 6 years ago Closed 6 years ago

Change JS-XMPP default to require encryption

Categories

(Chat Core :: XMPP, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: aleth, Assigned: aleth)

Details

Attachments

(1 file)

No description provided.
Attached patch requiretls.diffSplinter Review
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?
Assignee: nobody → aleth
Status: NEW → ASSIGNED
Attachment #8550461 - Flags: review?(clokep)
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+
SSL support is reported on https://xmpp.net/. Stats are https://xmpp.net/reports.php
Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 6 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.