Skip to content

test!: Path ////path causing failures #185

Description

@faretek1

The path ////path is causing issues with most of the test fixtures.

Image

https://github.com/gruns/furl/blob/f53defbf075e6f31c5b910aefb4221abe5b937cf/tests/test_furl.py#L1627C28-L1631C29

This is caused by the output of f.url == '//////path' instead of '////path'. This may be due to a change in the urllib library, but I am not sure.

The output of calling urllib.parse.unparse here:
https://github.com/gruns/furl/blob/f53defbf075e6f31c5b910aefb4221abe5b937cf/furl/furl.py#L1826C9-L1832C11

actually generates the expected number of slashes:

Image

However, because of this code, an extra 2 slashes are prepended, causing the output of f.url to be invalid.

Note

This issue does not appear on python 3.8, but appears on all other tests, so this could be a problem caused by a change in the standard library, in which case, other functionality may need reviewing

Metadata

Metadata

Assignees

Labels

No labels
No labels

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions