[FIX] base: don't overwrite existing extensions

The point was to fix files attachments without an extension,
potentially appending to existing extensions was not a smart idea.

See #34299, 5eb15c01b5
This commit is contained in:
Xavier Morel
2019-07-02 10:03:28 +02:00
parent 2f2bc67c06
commit 97f5e2ff0d
+5 -3
View File
@@ -346,9 +346,11 @@ class IrHttp(models.AbstractModel):
mimetype = guess_mimetype(base64.b64decode(content), default=default_mimetype)
# extension
extension = mimetypes.guess_extension(mimetype)
if extension and not filename.endswith(extension):
filename = "%s%s" % (filename, extension)
_, existing_extension = os.path.splitext(filename)
if not existing_extension:
extension = mimetypes.guess_extension(mimetype)
if extension:
filename = "%s%s" % (filename, extension)
headers += [('Content-Type', mimetype), ('X-Content-Type-Options', 'nosniff')]