Move HttpContentCompressorSubstitutions to quarkus-netty

[gastaldi]

Fixes #23218

[cescoffier]

Approved, but I want to be sure it fixes the reported issue.

[cescoffier]

@gastaldi I made two minor comments.

[cescoffier]

Something is odd...
We already have:

@TargetClass(CompressorHttp2ConnectionEncoder.class)
    public static final class CompressorHttp2ConnectionSubstitute {

        @Substitute
        protected EmbeddedChannel newContentCompressor(ChannelHandlerContext ctx, CharSequence contentEncoding)
                throws Http2Exception {
            if (GZIP.contentEqualsIgnoreCase(contentEncoding) || X_GZIP.contentEqualsIgnoreCase(contentEncoding)) {
                return newCompressionChannel(ctx, ZlibWrapper.GZIP);
            }
            if (DEFLATE.contentEqualsIgnoreCase(contentEncoding) || X_DEFLATE.contentEqualsIgnoreCase(contentEncoding)) {
                return newCompressionChannel(ctx, ZlibWrapper.ZLIB);
            }
            // 'identity' or unsupported
            return null;
        }

        @Alias
        private EmbeddedChannel newCompressionChannel(final ChannelHandlerContext ctx, ZlibWrapper wrapper) {
            throw new UnsupportedOperationException();
        }
    }

[cescoffier]

@gastaldi so, it's not the proper fix. We already have the included substitutions. They are in the quarkus-vertx-http (Runtime) module. I can't remember why there are there... However, we need to copy it to the netty runtime module.

[gastaldi]

Moved from vertx-http

[gsmet]

The commit message should be changed as it's pretty confusing now.

[gsmet]

Ah perfect, you just did it :).

[gastaldi]

The commit message should be changed as it's pretty confusing now.

yeap, good catch