-
Notifications
You must be signed in to change notification settings - Fork 189
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
make mimetype allow list configurable and add example config #2553
Conversation
Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes. |
72d8b89
to
85267eb
Compare
💥 Acceptance tests Core-API-Tests-ocis-storage-7 failed. The build is cancelled... |
0735e18
to
8fd705e
Compare
💥 Acceptance tests Core-API-Tests-ocis-storage-5 failed. The build is cancelled... |
760334d
to
e94565f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just style changes, other than that LGTM
@@ -13,6 +13,7 @@ node_modules/ | |||
ocis/ocis | |||
ocis/cmd/ocis/__debug_bin | |||
ocis/cmd/ocis/config/ | |||
config/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🙏 thanks
561e977
to
66f430b
Compare
Kudos, SonarCloud Quality Gate passed! |
Description
Since cs3org/reva#2095 mime types must be on an allow list to be advertised on the
/app/list
endpoint. Additionally one can now add some more information about mime types and select a default application per mime type as requested in #2563.This PR adds a basic default configuration and the possibility to configure the mime types with a config file.
Related Issue
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: