Rocket.chat: Default accepted file uploads

[aaronhagopian]

Running the latest docker image, I cannot upload Excel or CSV files. Not sure why the system is setup to reject any mime types (unless there is some security thoughts i'm missing).

I noticed an older issue where restarting fixed for them but i have restarted and the issue has not gone away.

[aaronhagopian]

Figured it out but seems like not sane defaults for people new to rocketchat. I suggest instead changing this issue to address common file types to be allowed by default like excel, zip etc.

[twhiting]

Status on this issue?

[engelgabriel]

What do you guys suggest for the default value?

[twhiting]

The reason I bring this up is we cannot upload .zip files even when the setting is specifically set to allow all types of uploads. This was marked as an improvement a month ago and seemed to be a dead issue. Thanks for adding it to the milestone @engelgabriel

Ideally, I think that when all types of uploads are allowed that should be taken literally to allow all types. Including ZIP / CSV / EXCEl etc. Then a separate toggle switch could be made to limit some hazardous file types if desired.

Edit: a field to list accepted ( or all ) filetypes and another to specify dis-allowed filetypes. This might be the best solution

[engelgabriel]

@twhiting do you mean the Accepted Media Types?

[twhiting]

Yes, as stated if I "Leave it blank for accepting all media types" I still cannot upload a zip file. Is this by design?

[engelgabriel]

Did you try restarting the server after changing the setting?

[twhiting]

Yes

[twhiting]

To add to this issue. I currently have my file upload set to (since leaving it blank doesn't work):

image/*,audio/*,application/*,text/*,video/*,*/*

And I still cannot upload file types such as: .sys .dmp .LOG .inf

[engelgabriel]

You can now leave it blank or just put a single "*" as on the settings to allow all.
This should be fixed on version 0.23.0
Please reopen otherwise.

[ulope]

This should really be explained better in the UI. Especially since there now seem to be two kinds of "all". Empty means some ill-defined subset of all while * means really-yes-I-want-all.

[engelgabriel]

Strange, that shouldn't be the case. Empty and * should be the same.

@sampaiodiego can you investigate this?

[souhaib22]

For CSV file add
text/csv or my configuration image/*,audio/*,application/zip,application/pdf,text/plain, text/csv,application/ms-powerpoint,application/msword,application/ms-excel,application/vnd.openxmlformats-officedocument.wordprocessingml.document,application/vnd.openxmlformats-officedocument.spreadsheetml.sheet,application/vnd.openxmlformats-officedocument.presentationml.presentation
Word(.doc & docx)
Excel(.xls & xls)
Powerpoint(.ppt & .pptx)

[lightweight]

Would be great to include Open Document formats by default, too... application/vnd.oasis.opendocument.*

[lightweight]

I note that manually specifying application/vnd.oasis.opendocument.* (and restarting the server) does not seem to successfully allow application/vnd.oasis.opendocument.text- am I misunderstanding how "*" (wildcards) work? I'm on 0.59.4

[andrewbrooke72]

if you have a popup saying that text/csv not allowed.. add it to access media types with comma at the end of the line ",text/csv"

[igor9silva]

I'm not able to allow upload of .plist files. I've tried adding plist, .plist, *.plist, */plist and application/x-plist. The only way I could find out was leaving the whole field blank.

[vynmera]

@igor9silva
Have you tried
application/xml or application/octet-stream?

[igor9silva]

@vynmera application/xml was already there. I've just tried application/octet-stream which failed as well.