Hello Python-Committers,

In https://bugs.python.org/issue27657, I introduced a regression in a minor
release.
The original patch to parsing logic of URL, cleaned up a lot of corner
cases (https://github.com/python/cpython/pull/661) and I felt good about
the change.
However, the mistake was with the backport.

Demo:

$ ./python
Python 3.8.0 (default, Feb 10 2020, 06:15:43)
[GCC 9.2.1 20191008] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from urllib.parse import urlparse
>>> urlparse('localhost:8080')
ParseResult(scheme='', netloc='', path='localhost:8080', params='',
query='', fragment='')

----

$ ./python
Python 3.8.1+ (heads/3.8:b086ea5edc, Feb 10 2020, 06:15:44)
[GCC 9.2.1 20191008] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from urllib.parse import urlparse
>>> urlparse('localhost:8080')
ParseResult(scheme='localhost', netloc='', path='8080', params='',
query='', fragment='')

---

When I read an associated bug report against a user of the software (like
this: https://github.com/mozilla/bleach/issues/503) - I feel that this was
a mistake.
The change of test-suite in minor versions should have alerted me, but I
seem to have missed it.

I am planning to revert this change in 3.8.2 and 3.7.7
Should I highlight this in any documentation?  Thoughts and opinions?

Thank you,
Senthil
_______________________________________________
python-committers mailing list -- python-committers@python.org
To unsubscribe send an email to python-committers-le...@python.org
https://mail.python.org/mailman3/lists/python-committers.python.org/
Message archived at 
https://mail.python.org/archives/list/python-committers@python.org/message/2ONL562L35S77FGCLYH2ZDWVAXHH2BZP/
Code of Conduct: https://www.python.org/psf/codeofconduct/

Reply via email to