Terry J. Reedy added the comment: Are you talking about top-level code within the urllib module or code within defined functions. If the former, can you quote or point to the place in the file? If the latter, which functions? Just urlopen or others? Does urllib2.urlopen have the same issue? (That replaces urllib.urlopen in 3.x). Having answered the above, please suggest a specific change in a specific place in the docs. ---------- nosy: +terry.reedy stage: -> needs patch versions: +Python 2.7 -Python 2.6 _______________________________________ Python tracker http://bugs.python.org/issue9637 _______________________________________

Kirikaza added the comment: I missed the print statement in that example... So I attached the file with correct code. ---------- Added file: http://bugs.python.org/file18594/proxy.py _______________________________________ Python tracker http://bugs.python.org/issue9637 _______________________________________

Neil Muller added the comment: The problem is that the getproxies_environment function converts all environment variables to lower case before deciding whether to use the name. This means that whichever ends up last in os.environment will be used. The attached patch does two things. It changes the behaviour to prefer the all lower case version if it exists, and adds a note to the ProxyHandler description noting that the environment variable used is not case sensitive, although the all lower case version will be preferred. The behaviour change is debatable, but preferring the lower case name is likely to be less surprising in general. ---------- keywords: +patch nosy: +Neil Muller Added file: http://bugs.python.org/file19678/urllib_PROXY.diff _______________________________________ Python tracker http://bugs.python.org/issue9637 _______________________________________

Terry J. Reedy added the comment: Senthil, any comment on this? ---------- nosy: +orsenthil _______________________________________ Python tracker http://bugs.python.org/issue9637 _______________________________________

Roundup Robot added the comment: New changeset 3370fa13ed73 by Senthil Kumaran in branch '3.2': Issue9637 - Explain in getproxies_environment that <scheme>_proxy environ variable case does not matter. http://hg.python.org/cpython/rev/3370fa13ed73 New changeset eb028b3c62c9 by Senthil Kumaran in branch 'default': Merge from 3.2 - Issue9637 - Explain in getproxies_environment that <scheme>_proxy environ variable case does not matter. http://hg.python.org/cpython/rev/eb028b3c62c9 ---------- nosy: +python-dev _______________________________________ Python tracker http://bugs.python.org/issue9637 _______________________________________

Senthil Kumaran added the comment: This is fixed. Thanks for your contribution. ---------- resolution: -> fixed stage: needs patch -> committed/rejected status: open -> closed _______________________________________ Python tracker http://bugs.python.org/issue9637 _______________________________________