Martin Panter added the comment:

I do wonder if NotImplementedError is the right exception. According to the 
documentation it is derived from RuntimeError and is meant for abstract 
methods, i.e. it is a programmer error. Other cases in urllib.request raise 
ValueError (e.g. AbstractBasicAuthHandler), which is fairly normal for protocol 
errors like this.

Also it would be nice to add a test case for the bug fix.

----------
nosy: +martin.panter
stage: needs patch -> patch review
versions: +Python 2.7, Python 3.5, Python 3.6

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue2202>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to