Closed Bug 741536 Opened 12 years ago Closed 12 years ago

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

Categories

(Thunderbird :: Instant Messaging, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(thunderbird15 fixed, thunderbird16 fixed)

RESOLVED FIXED
Thunderbird 17.0
Tracking Status
thunderbird15 --- fixed
thunderbird16 --- fixed

People

(Reporter: BenB, Assigned: florian)

References

()

Details

Attachments

(1 file)

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.
possibly bug 741056 but that's a crasher on windows.
Summary: IRC fails when there's a system HTTP proxy → IRC and XMPP connections fail when there's a system HTTP proxy
Attached patch PatchSplinter Review
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.
Assignee: nobody → florian
Attachment #647488 - Flags: review?(clokep)
Comment on attachment 647488 [details] [diff] [review]
Patch

This looks fine. Thanks.
Attachment #647488 - Flags: review?(clokep) → review+
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.
Attachment #647488 - Flags: approval-comm-beta?
Attachment #647488 - Flags: approval-comm-aurora?
> 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.
Attachment #647488 - Flags: approval-comm-beta?
Attachment #647488 - Flags: approval-comm-beta+
Attachment #647488 - Flags: approval-comm-aurora?
Attachment #647488 - Flags: approval-comm-aurora+
You need to log in before you can comment on or make changes to this bug.