Comment 5 for bug 601551

Revision history for this message
Martin Packman (gz) wrote :

Wrapping in try/finally is what I tried, which indeed doesn't really feel like right thing. Also problematic are that this is in bzrlib.transport.http._urllib so presumably pycurl will need some kind of change as well, and this looks a pain to test. Think it really wants a blackbox style test to ensure the console interactions are sane, but doing that across http libraries and authentication methods, which avoiding leaking server threads, looks complicated.