@@ -67,7 +67,7 @@ def nameext_from_url(url, data=None):
|
|||||||
|
|
||||||
filename = unquote(filename_from_url(url))
|
filename = unquote(filename_from_url(url))
|
||||||
name, _, ext = filename.rpartition(".")
|
name, _, ext = filename.rpartition(".")
|
||||||
if name:
|
if name and len(ext) <= 16:
|
||||||
data["filename"], data["extension"] = name, ext.lower()
|
data["filename"], data["extension"] = name, ext.lower()
|
||||||
else:
|
else:
|
||||||
data["filename"], data["extension"] = filename, ""
|
data["filename"], data["extension"] = filename, ""
|
||||||
|
|||||||
@@ -148,6 +148,10 @@ class TestText(unittest.TestCase):
|
|||||||
self.assertEqual(
|
self.assertEqual(
|
||||||
f("http://example.org/v2/filename.ext?param=value#frag"), result)
|
f("http://example.org/v2/filename.ext?param=value#frag"), result)
|
||||||
|
|
||||||
|
# long "extension"
|
||||||
|
fn = "httpswww.example.orgpath-path-path-path-path-path-path-path"
|
||||||
|
self.assertEqual(f(fn), {"filename": fn, "extension": ""})
|
||||||
|
|
||||||
# invalid arguments
|
# invalid arguments
|
||||||
for value in INVALID:
|
for value in INVALID:
|
||||||
self.assertEqual(f(value), empty)
|
self.assertEqual(f(value), empty)
|
||||||
|
|||||||
Reference in New Issue
Block a user