741536 - IRC and XMPP connections fail when there's a system HTTP proxy

Status: RESOLVED FIXED

(Thunderbird :: Instant Messaging, defect)

Description

[Ben Bucksch (:BenB)]

Reproduction:
1. In GNOME settings | Proxy, enter a HTTP proxy.
2. Create a new TB profile, and start TB from the commandline
3. Create an IRC account, and connect

Actual result:
"Your chat accounts are not connected".
"Show chat status" shows "Lost connection to server".

Error console shows:
irc: Unhandled IRC message: HTTP/1.0 400 Bad Request
irc: Unhandled IRC message: Server: squid/...
irc: Unhandled IRC message: Date: Mon, 02 Apr 2012 20:15:07 GMT
irc: Unhandled IRC message: Content-Type: text/html
irc: Unhandled IRC message: Content-Length: 1221
irc: Unhandled IRC message: Expires: Mon, 02 Apr 2012 20:15:07 GMT
irc: Unhandled IRC message: X-Squid-Error: ERR_INVALID_REQ 0
irc: Unhandled IRC message: X-Cache: MISS from squid.host
irc: Unhandled IRC message: X-Cache-Lookup: NONE from squid.host:1234
irc: Unhandled IRC message: Connection: close

Expected result:
No HTTP proxy used for IRC.

Comment 1

[Joe Sabash [:JoeS1]]

possibly bug 741056 but that's a crasher on windows.

Comment 2

[Patrick Cloke [:clokep]]

We have a similar Instantbird bug on file: https://bugzilla.instantbird.org/show_bug.cgi?id=1347

Comment 4

[Florian Quèze [:florian]]

Attachment: Patch

I'm proposing with this patch that we just ignore HTTP proxies for now.

I know it should be possible to use some HTTP proxies to tunnel a non-HTTP connection, but I don't know exactly what we need to do for that, and I've no HTTP proxy supporting that to test with, so for now just ignoring the HTTP proxies seems the best thing to do.

Comment 5

[Patrick Cloke [:clokep]]

Comment on attachment 647488 [details] [diff] [review]
Patch

This looks fine. Thanks.

Comment 6

[Florian Quèze [:florian]]

Comment on attachment 647488 [details] [diff] [review]
Patch

[Approval Request Comment]
I've seen several reports of people who couldn't connect there chat accounts in Thunderbird because of this, so I think we should take this for Thunderbird 15.

Comment 7

[Ben Bucksch (:BenB)]

> I'm proposing with this patch that we just ignore HTTP proxies for now.

+1

This also allows to use the trick mentioned in bug 741550.

Comment 8

[Florian Quèze [:florian]]

https://hg.mozilla.org/comm-central/rev/b9acacd77af4
https://hg.mozilla.org/releases/comm-aurora/rev/ebababc57916
https://hg.mozilla.org/releases/comm-beta/rev/37202ee972f6