829597 - Proxy connections completely broken by bug 769764

Status: RESOLVED FIXED

(Other Applications :: ChatZilla, defect)

Description

[James Ross]

Bug 769764 removes all means of synchronously resolving URLs to proxy data, which naturally breaks ChatZilla's use of proxies (and only by lucky coding doesn't break all connections).

Comment 1

[James Ross]

Attachment: Fix some minor DCC bugs

This just fixes the display of DCC messages to show the remoteIP instead of localIP, making it match /dcc-list and the correct expectations, and the /dcc-list summary mislabelled failed as finished connections.

Comment 2

[James Ross]

Attachment: Fix the connections to be async

The real deal; change CBSConnection's connect method to be async in the face of proxies (and sync callback otherwise). Also includes some more DCC tidy-up of unused properties.

Comment 3

[Simon Lucy]

This bug has quite crippled internal BBC operations comms with users stuck on Firefox 17, hence my vote.

Comment 4

[James Ross]

Attachment: Updated patch to fix /cancel bugs

This update fixes /cancel during the proxy lookup phase. If you /cancel during the proxy lookup phase, the cancelling message appears and then when it completes, the connection is cancelled.

Comment 5

[James Ross]

http://hg.mozilla.org/chatzilla/file/4372bfb647e1
http://hg.mozilla.org/chatzilla/file/ff4fb7cb46c9

Simon Lucy, in about 45 minutes an automated build with these patches should appear on http://james-ross.co.uk/mozilla/chatzilla/nightly/ - please do let us know if this improves the situation at the BBC.

Likewise, anyone else using proxy connections, please report back if the nightly build fixes or doesn't fix the issue.

Comment 6

[Simon Lucy]

(In reply to James Ross from comment #5)
> http://hg.mozilla.org/chatzilla/file/4372bfb647e1
> http://hg.mozilla.org/chatzilla/file/ff4fb7cb46c9
> 
> Simon Lucy, in about 45 minutes an automated build with these patches should
> appear on http://james-ross.co.uk/mozilla/chatzilla/nightly/ - please do let
> us know if this improves the situation at the BBC.
> 
> Likewise, anyone else using proxy connections, please report back if the
> nightly build fixes or doesn't fix the issue.

That url 404s for me I'm afraid James.

Comment 7

[Simon Lucy]

(In reply to Simon Lucy from comment #6)
> (In reply to James Ross from comment #5)
> > http://hg.mozilla.org/chatzilla/file/4372bfb647e1
> > http://hg.mozilla.org/chatzilla/file/ff4fb7cb46c9
> > 
> > Simon Lucy, in about 45 minutes an automated build with these patches should
> > appear on http://james-ross.co.uk/mozilla/chatzilla/nightly/ - please do let
> > us know if this improves the situation at the BBC.
> > 
> > Likewise, anyone else using proxy connections, please report back if the
> > nightly build fixes or doesn't fix the issue.
> 
> That url 404s for me I'm afraid James.

Ignore that something peculiar in a different proxy, fine with Chrome

Comment 8

[Simon Lucy]

Thanks very much I thought I'd commented after testing but evidently not,