[issue16300] Windows raises ValueError for file:///file/not/exists instead of URLError

[Senthil Kumaran]

Senthil Kumaran added the comment:

Buildbot issues are taken care. They are green for while. Tests have been 
backported 3.3 and 3.2 as well.

3.2 5e71f2712076
3.3 30547e2cd04d

--
resolution:  - fixed
stage:  - committed/rejected
status: open - closed

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16300
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue16300] Windows raises ValueError for file:///file/not/exists instead of URLError

[Roundup Robot]

Roundup Robot added the comment:

New changeset 00e5e963b7d8 by Senthil Kumaran in branch 'default':
Fix issue16300: addressing the buildbot failures on windows
http://hg.python.org/cpython/rev/00e5e963b7d8

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16300
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue16300] Windows raises ValueError for file:///file/not/exists instead of URLError

[Senthil Kumaran]

New submission from Senthil Kumaran:

From the Buildbot failure - 
http://buildbot.python.org/all/builders/x86%20XP-4%203.x/builds/7573/steps/test/logs/stdio

I remember that we restricted the access for file:// scheme to localhost only 
for Windows/Cygwin. In terms of exception raised it could either be consistent 
with other platforms and raise URLError or it could be for good reasons, the 
tests may need to be tweaked for windows.

--
assignee: orsenthil
keywords: easy
messages: 173522
nosy: orsenthil
priority: normal
severity: normal
status: open
title: Windows raises ValueError for file:///file/not/exists instead of URLError
type: behavior
versions: Python 3.2, Python 3.3, Python 3.4

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16300
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue16300] Windows raises ValueError for file:///file/not/exists instead of URLError

[Senthil Kumaran]

Senthil Kumaran added the comment:

The problem is here:

if url[:2] == '//' and url[2:3] != '/' and url[2:12].lower() != 'localhost/':
raise ValueError(file:// scheme is supported only on localhost)

On Unix like systems url[2:3] == '/', that is path starts with '/' and this 
ValueError is escaped.

--

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16300
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue16300] Windows raises ValueError for file:///file/not/exists instead of URLError

[Roundup Robot]

Roundup Robot added the comment:

New changeset 49de26395d1a by Senthil Kumaran in branch 'default':
Issue #16301: Fix the localhost verification in urllib/request.py for file://. 
Modify tests to use localhost for local temp files, which could make Windows 
Buildbot (#16300) happy
http://hg.python.org/cpython/rev/49de26395d1a

--
nosy: +python-dev

___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16300
___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com