search

OpenAPITools / openapi-generator

OpenAPI Generator allows generation of API client libraries (SDK generation), server stubs, documentation and configuration automatically given an OpenAPI Spec (v2, v3)
https://openapi-generator.tech
Apache License 2.0
22.03k stars 6.61k forks source link

[REQ] [SERVER] [Java] [Quarkus] useGzipSupport is not implemented #13904

Open coiouhkc opened 2 years ago

coiouhkc commented 2 years ago

Is your feature request related to a problem? Please describe.

pom.xml

<configOptions>
                <generateSupportingFiles>false</generateSupportingFiles>
                <library>quarkus</library>
                <serializationLibrary>jackson</serializationLibrary>
                <dateLibrary>java8</dateLibrary>
                <useBeanValidation>false</useBeanValidation>
                <useGzipFeature>true</useGzipFeature>
                <generatePom>false</generatePom>
                <interfaceOnly>true</interfaceOnly>
                <generateBuilders>true</generateBuilders>
                <useSwaggerAnnotations>false</useSwaggerAnnotations>
              </configOptions>

but generated interfaces do not include @GZIP annotation.

Describe the solution you'd like

Generated interfaces should include the @GZIP annotation, application.properties must include necessary configuration for both classic and reactive Quarkus modes, see https://quarkus.io/guides/resteasy#gzip-support

Additional context

@Override + @GZIP in the implementing class unfortunately doesn't work.

coiouhkc commented 2 years ago

I'm happy to work on a PR if you generally accept the feature request.

wing328 commented 2 years ago

@coiouhkc yes, we welcome PR to add useGzipFeature support to the generator.

Let us know if you need help with the PR.

coiouhkc commented 2 years ago

Started work in https://github.com/OpenAPITools/openapi-generator/pull/13983

coiouhkc commented 1 year ago

Please resolve the issue, since the linked PR has been merged.