Support for GZIP in GraphQL Services

[funcbro]

Description

The GraphQL extension io.quarkus:quarkus-smallrye-graphql currently does not support GZIP compression of responses.

GZIP compression of GraphQL-responses should be configurable.

When looking at the smallrye implementation itself, it should be possible to implement a GZIP compression via a Servlet Response-Filter.
By using the quarkus extension though this possibility does not seem to exist.

Implementation ideas

No response

[quarkus-bot]

/cc @jmartisk, @phillip-kruger

[phillip-kruger]

Thanks for this issue. The Quarkus extension does not use servlet for the HTTP transport. But we should be able to add this to the vertx route. I'll have a look a.s.a.p.

[funcbro]

any news on this issue?

[phillip-kruger]

No news yet. You are welcome to do a PR if you want. The change would probably be somewhere in the Quarkus GraphQL Extension Runtime

[phillip-kruger]

Also add support for https://github.com/quarkusio/quarkus/blob/main/extensions/vertx-http/runtime/src/main/java/io/quarkus/vertx/http/Compressed.java

[www-wanglong]

I just need it.

[gsmet]

If we are using a Vert.x route, shouldn’t this be already supported with @mkouba ’s work?

[mkouba]

If we are using a Vert.x route, shouldn’t this be already supported with @mkouba ’s work?

I don't think so. IIRC the compression support was not added to Vert.x routes in general but into Reactive Routes, RESTEasy Reactive, Undertow Servlet, RESTEasy Classsic and static resources. More specifically, if quarkus.http.enable-compression=true then we add the Content-Encoding: identity header that disables the compression to every response and the task of the support is to remove this header if appropriate.