[issue34628] urllib.request.urlopen fails when userinfo is present in URL

Change by Irit Katriel : -- versions: +Python 3.10, Python 3.11, Python 3.9 -Python 3.4, Python 3.5, Python 3.6, Python 3.7 ___ Python tracker ___

[issue34628] urllib.request.urlopen fails when userinfo is present in URL

Windson Yang added the comment: Why requests library didn't raise an error because urllib3 (the library requests using) ignore the auth part right now > Currently we expect our users to handle authentication headers themselves. > It's unfortunate that we silently strip this information

[issue34628] urllib.request.urlopen fails when userinfo is present in URL

Windson Yang added the comment: I found that Requests library use urllib3 library which looks like ignore the user info part (in request_context https://github.com/urllib3/urllib3/blob/master/src/urllib3/poolmanager.py#L208). Did I miss something or we should also ignore it? --

[issue34628] urllib.request.urlopen fails when userinfo is present in URL

Niklas Sombert added the comment: This behaviour would be better than the current one, yes. -- ___ Python tracker ___ ___

[issue34628] urllib.request.urlopen fails when userinfo is present in URL

Windson Yang added the comment: First, we can add some check at https://github.com/python/cpython/blob/e42b705188271da108de42b55d9344642170aa2b/Lib/http/client.py#L871 and raise an error if the URL contains userinfo part. Second, we should catch some exception in urllib.request.urlopen, if

[issue34628] urllib.request.urlopen fails when userinfo is present in URL

Niklas Sombert added the comment: Another month has passed, just saying. -- ___ Python tracker ___ ___ Python-bugs-list mailing

[issue34628] urllib.request.urlopen fails when userinfo is present in URL

Niklas Sombert added the comment: Hey, this is almost a month old. (Not a problem, really. But I thought, I should bump this.) -- ___ Python tracker ___

[issue34628] urllib.request.urlopen fails when userinfo is present in URL

Niklas Sombert added the comment: Thank you for your response. But if this is an explicit choice, I would like to have better exceptions: >>> from http.client import HTTPConnection >>> h = HTTPConnection("user:1...@example.net") raises http.client.InvalidURL: nonnumeric port:

[issue34628] urllib.request.urlopen fails when userinfo is present in URL

Karthikeyan Singaravelan added the comment: Seems like this is an explicit choice that there is also a test with the URL "user:passw...@www.python.org" . Ref : https://github.com/python/cpython/blob/731ff68eeef58babdf2b32dc9a73b141760c2be9/Lib/test/test_httplib.py#L640 . You can try basic

[issue34628] urllib.request.urlopen fails when userinfo is present in URL

Change by Karthikeyan Singaravelan : -- nosy: +xtreak ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue34628] urllib.request.urlopen fails when userinfo is present in URL

New submission from Niklas Sombert : Today I tried to access URLs like this one: http://user:1...@example.net:8080. The result was this: >>> import urllib.request >>> urllib.request.urlopen("http://user:1...@example.net:1234/;) Traceback (most recent call last): File