[issue40938] urllib.parse.urlunsplit makes relative path to absolute (http:g -> http:///g)

[Open Close]

Open Close  added the comment:

I tried hard (even read RFC1630),
but I think no.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40938] urllib.parse.urlunsplit makes relative path to absolute (http:g -> http:///g)

[Jonathan Schweder]

Jonathan Schweder  added the comment:

Not exactly, in the RFC example they use a/b/c for the path, but when using 
http:g there is no nested path, so it should be http:///g, no?

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40938] urllib.parse.urlunsplit makes relative path to absolute (http:g -> http:///g)

[Open Close]

Open Close  added the comment:

'http:///g' has absolute path '/g',
and as urljoin shows:

>>> urljoin('http://a/b/c/d', 'http:///g')
'http://a/g'  # 'a' is netloc

So you are proposing third interpretation.

  "http:g"=  "http:g" ; for strict parsers
  /  "http://a/b/c/g; ; for backward compatibility
  /  "http://a/g; ; (yours)

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40938] urllib.parse.urlunsplit makes relative path to absolute (http:g -> http:///g)

[Open Close]

Open Close  added the comment:

hello, @jaswdr, but I can't understand what's wrong with my point.
What is 'the expected behaviour'?

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40938] urllib.parse.urlunsplit makes relative path to absolute (http:g -> http:///g)

[hai shi]

Change by hai shi :


--
nosy: +orsenthil

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40938] urllib.parse.urlunsplit makes relative path to absolute (http:g -> http:///g)

[Jonathan Schweder]

Jonathan Schweder  added the comment:

@op368 I don't think that this is a bug, [1] literally uses this exact example 
and shows the expected behaviour. 


[1] https://datatracker.ietf.org/doc/html/rfc3986#section-5.4.2

--
nosy: +jaswdr

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40938] urllib.parse.urlunsplit makes relative path to absolute (http:g -> http:///g)

[wyz23x2]

Change by wyz23x2 :


--
versions: +Python 3.7, Python 3.9

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40938] urllib.parse.urlunsplit makes relative path to absolute (http:g -> http:///g)

[Open Close]

Change by Open Close :


--
components: +Library (Lib)

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue40938] urllib.parse.urlunsplit makes relative path to absolute (http:g -> http:///g)

[Open Close]

New submission from Open Close :

path 'g' in 'http:g' becomes '/g'.

>>> urlsplit('http:g')
SplitResult(scheme='http', netloc='', path='g', query='', fragment='')
>>> urlunsplit(urlsplit('http:g'))
'http:///g'
>>> urlsplit('http:///g')
SplitResult(scheme='http', netloc='', path='/g', query='', fragment='')

>>> urljoin('http://a/b/c/d', 'http:g')
'http://a/b/c/g'
>>> urljoin('http://a/b/c/d', 'http:///g')
'http://a/g'

The problematic part of the code is:

def urlunsplit(components):
[...]
if netloc or (scheme and scheme in uses_netloc and url[:2] != '//'):
--->if url and url[:1] != '/': url = '/' + url
url = '//' + (netloc or '') + url

Note also that urllib has decided on the interpretation of 'http:g' (in test).

def test_RFC3986(self):
[...]
#self.checkJoin(RFC3986_BASE, 'http:g','http:g') # strict parser
self.checkJoin(RFC3986_BASE, 'http:g','http://a/b/c/g') #relaxed parser

--
messages: 371179
nosy: op368
priority: normal
severity: normal
status: open
title: urllib.parse.urlunsplit makes relative path to absolute (http:g -> 
http:///g)
type: behavior
versions: Python 3.8

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com