[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:
@@ -321,8 +321,9 @@ class IrHttp(models.AbstractModel):
|
||||
mimetype = guess_mimetype(base64.b64decode(content), default=default_mimetype)
|
||||
|
||||
# extension
|
||||
_, existing_extension = os.path.splitext(filename)
|
||||
extension = mimetypes.guess_extension(mimetype)
|
||||
if extension and not filename.endswith(extension):
|
||||
if not existing_extension and extension and not filename.endswith(extension):
|
||||
filename = "%s%s" % (filename, extension)
|
||||
|
||||
headers += [('Content-Type', mimetype), ('X-Content-Type-Options', 'nosniff')]
|
||||
|
||||
Reference in New Issue
Block a user