-
Notifications
You must be signed in to change notification settings - Fork 19
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
Add binary-configuration to servlet-core #504
Conversation
This class was used in both AWS and Azure, so we have pulled it up into servlet-core
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.
It misses a test
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.
I added a unit test
SonarCloud Quality Gate failed. 0 Bugs 0.0% Coverage Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
I think more about this I think we should do this in core and it should be probably configurable. For example: A user may want to set:
to add extra types to the list we provide by default. |
This is used by the AWS and Azure modules. Originally we moved this duplication to Servlet-core: micronaut-projects/micronaut-servlet#504 But it was then decided that core was a better home for it. It specified defaults, and allows users to ignore these defaults, and add their own mime types.
Superseded by micronaut-projects/micronaut-core#9530 |
This class was used in both AWS and Azure, so we have pulled it up into servlet-core