Add BinaryTypeConfiguration

[timyates]

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.

[graemerocher]

but it isn't integrated with the Netty server and client?

[timyates]

@graemerocher we ended up with the same thing in Aws and Azure. First, we moved it to servlet-core here micronaut-projects/micronaut-servlet#504 but then it was decided that it should move to core.

But it's not used in netty (server or client)

Finding a home for things is hard ☹️

[sdelamo]

add missing package-info.java file

[sdelamo]

@graemerocher and @timyates I think this belongs in micronaut-http it is related with media types. It make sense, we place it in the module where MediaType.java is.

[graemerocher]

I don't think this should be merged if it is not integrated with Netty, it will just be confusing

[sdelamo]

I don't think this should be merged if it is not integrated with Netty, it will just be confusing

Do you think we should add it servlet-core or duplicate it across the serverless modules? For me, micronaut-http should serve every http implementation not just netty.

[graemerocher]

if it is specific to server less probably it makes sense in the function module. If you add it to servlet-core then the same question applies, will it be applied to Jetty/Tomcat/Undertow?

[sdelamo]

@timyates Please, check if every usage of BinaryTypeConfiguratino is an a module which current depends on io.micronaut:micronaut-function

[timyates]

@sdelamo

Azure

Azure does as it has an api dependency on azure-function which has an api dependency on micronaut-function

AWS

And AWS does the same

I'll move this into micronaut-function 👍

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

81.8% Coverage
0.0% Duplication

[sdelamo]

@graemerocher can you review this ?

[sdelamo]

ping @graemerocher

[sdelamo]

@timyates can you create prs in the different modules to use this API.