[issue22586] urljoin allow_fragments doesn't work

2014-10-12 Thread Roundup Robot
Roundup Robot added the comment: New changeset 9eed2e7fa764 by Georg Brandl in branch '3.4': Closes #22586: clarify meaning of allow_fragments in urlparse. https://hg.python.org/cpython/rev/9eed2e7fa764 New changeset c2eda29a8ccb by Georg Brandl in branch '2.7': Closes #22586: clarify meaning of

[issue22586] urljoin allow_fragments doesn't work

2014-10-09 Thread Georg Brandl
Georg Brandl added the comment: The "not allowed" should be clarified. What is meant is that if allow_fragments is false, a fragment is parsed as part of the path. This doesn't make a difference for urljoin if the fragment is part of the second part. It does make a difference for the first p

[issue22586] urljoin allow_fragments doesn't work

2014-10-09 Thread Alex Parrill
New submission from Alex Parrill: Passing False to the allow_fragments argument to urljoin doesn't remove fragments. Is this a bug, or am I misunderstanding the allow_fragments parameter? It's not perfectly clear what "fragment identifiers are not allowed" means (strips them out? throws an er