url_downloadable does not respect proxy

I'm seeing this behavior too. I was trying to update an Ubuntu Server to test Karmic and spotted this. It was very weird because "sudo do-release-upgrade" apparently works and "sudo do-release-upgrade -d" doesn't.

I checked the process environment and the $http_proxy variable is correctly defined. Then I started fiddling with the python script and found this bug.

I solved the problem and tested it on my system. The upgrade to a development release now works fine.

I use urllib2 to test the repository url insted of using httplib and ftplib, so the proxy definitions are used. The code is more compact too.

Can someone have a look?

At the end of line 48 it is missing a :
I tried that and still not working, I will try to build a opener with proxy.

After adding the patch in comment #2 and the fix comment #2, it works for me.

Quite sad that this bug was not fixed in one month after the patch was submitted :'(

Thanks for the bugreport and the patch. The patch is not ideal because urllib2 always uses GET (for http) and RETR (for ftp). This is not ideal as it will cause more traffic than needed, especially when looking for a big File like Packages.gz. This is why the (more complicated) version with httplib/ftplib is used. In trunk this should be fixed and proxy support should be fully working.

  * UpdateManager/Core/utils.py:
    - fix url_downloadable and add tests, based on the patch from
      Paulo Albuquerque, many thanks (LP: #396187)

This bug was fixed in the package update-manager - 1:0.141

---------------
update-manager (1:0.141) maverick; urgency=low

  * UpdateManager/backend/__init__.py:
    - switch to aptdaemon as install backend by default (unless
      the user has UPDATE_MANAGER_FORCE_BACKEND_SYNAPTIC in his
      environment)
    - merged lp:~glatzor/update-manager/ubuntu-glatzor (many thanks!)
  * DistUpgrade/DistUpgradeController.py:
   - use pockets from DistUpgrade.cfg instead of hard-coding them
  * tests/test_sources_list.py:
    - update tests
  * UpdateManager/Core/utils.py:
    - fix url_downloadable and add tests, based on the patch from
      Paulo Albuquerque, many thanks (LP: #396187)
  * UpdateManager/UpdateManager.py:
    - fix typo (thanks to seb128)
  * DistUpgrade/mirrors.cfg:
    - support upgrades when sources.list uses the new
      mirror://mirrors.ubuntu.com/mirrors.txt
      uri
  * UpdateManager/GtkProgress.py:
    - do not open a cache open progress window, instead show the
      progress inline in the window
 -- Michael Vogt <email address hidden> Mon, 31 May 2010 15:14:22 +0200

Accepted update-manager into lucid-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

SRU verification for Lucid:
I have reproduced the problem with update-manager 1:0.134.9 in lucid-updates and have verified that the version of update-manager 1:0.134.10 in -proposed fixes the issue. I've tested to the stage where update-manager starts to download the packages.

Marking as verification-done

This bug was fixed in the package update-manager - 1:0.134.10

---------------
update-manager (1:0.134.10) lucid-proposed; urgency=low

  * DistUpgrade/DistUpgradeMain.py:
    - add filter for "apt API not stable yet" warning (for
      hardy->lucid upgrades) (LP: #534200)
  * UpdateManager/Core/utils.py:
    - fix url_downloadable and add tests, based on the patch from
      Paulo Albuquerque, many thanks (LP: #396187)
  * DistUpgrade/DistUpgradeViewNonInteractive.py:
    - make the dpkg-status file parser more robust against
      unexpected data (LP: #574837)
 -- Michael Vogt <email address hidden> Wed, 30 Jun 2010 16:01:01 +0200