Skip to content
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 application/zip as a binary type and allow configuration #1754

Merged
merged 1 commit into from
Jun 6, 2023

Conversation

timyates
Copy link
Contributor

@timyates timyates commented Jun 6, 2023

Previously when we used serverless, we added application/zip to the list of binary types.

This was lost when we moved away from serverless, and was caught by a test in starter

micronaut-projects/micronaut-starter#1853

This PR changes from a static set of binary types to a singleton bean that allows types to be added to the set. It also adds application/zip as a binary type by default.

Previously when we used serverless, we added application/zip to the list of binary types.

This was lost when we moved away from serverless, and was caught by a test in starter

micronaut-projects/micronaut-starter#1853

This PR changes from a static set of binary types to a singleton bean that allows types to be
added to the set.  It also adds application/zip as a binary type by default.
@timyates timyates added the type: bug Something isn't working label Jun 6, 2023
@timyates timyates requested a review from sdelamo June 6, 2023 12:22
@timyates timyates self-assigned this Jun 6, 2023
@sonarcloud
Copy link

sonarcloud bot commented Jun 6, 2023

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@sdelamo sdelamo merged commit 0fce0fa into master Jun 6, 2023
@sdelamo sdelamo deleted the binary-types branch June 6, 2023 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working
Projects
No open projects
Status: Done
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants