From: Kenneth Reitz Date: Wed, 30 Oct 2013 19:32:19 +0000 (-0400) Subject: Revert "Merge pull request #1710 from jvantuyl/master" X-Git-Tag: v2.1.0~29 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0228851f19aaf4b0f620c74cb08d295329431a3e;p=services%2Fpython-requests.git Revert "Merge pull request #1710 from jvantuyl/master" This reverts commit a8300c0ae6dd7e6b1655cf80b349e0bf645ff694, reversing changes made to 2e196be143035fa6d03743d6ca95a2904572f12a. --- diff --git a/requests/models.py b/requests/models.py index a9ccaf7..a67868e 100644 --- a/requests/models.py +++ b/requests/models.py @@ -327,11 +327,6 @@ class PreparedRequest(RequestEncodingMixin, RequestHooksMixin): raise MissingSchema("Invalid URL {0!r}: No schema supplied. " "Perhaps you meant http://{0}?".format(url)) - # Don't do any URL preparation for oddball schemes - if scheme.lower() not in ('http', 'https'): - self.url = url - return - if not host: raise InvalidURL("Invalid URL %r: No host supplied" % url) diff --git a/requests/packages/urllib3/util.py b/requests/packages/urllib3/util.py index 19b7c7e..cf934d4 100644 --- a/requests/packages/urllib3/util.py +++ b/requests/packages/urllib3/util.py @@ -365,8 +365,6 @@ def parse_url(url): # Scheme if '://' in url: scheme, url = url.split('://', 1) - elif ':' in url: - scheme, url = url.split(':', 1) # Find the earliest Authority Terminator # (http://tools.ietf.org/html/rfc3986#section-3.2) diff --git a/test_requests.py b/test_requests.py index a67be62..a593db3 100755 --- a/test_requests.py +++ b/test_requests.py @@ -678,14 +678,6 @@ class RequestsTestCase(unittest.TestCase): assert p.headers['Content-Length'] == length - def test_oddball_schemes_dont_check_URLs(self): - r1 = requests.Request('GET', 'data:image/gif;base64,R0lGODlhAQABAHAAACH5BAUAAAAALAAAAAABAAEAAAICRAEAOw==') - r1.prepare() - r2 = requests.Request('GET', 'file:///etc/passwd') - r2.prepare() - r3 = requests.Request('GET', 'magnet:?xt=urn:btih:be08f00302bc2d1d3cfa3af02024fa647a271431') - r3.prepare() - class TestContentEncodingDetection(unittest.TestCase): def test_none(self):