Violeta Georgieva

Results 263 comments of Violeta Georgieva

> [@violetagg](https://github.com/violetagg) Hello, > > I'm wondering if there are any plans to support this feature in Reactor Netty. If so, would it be alright for me to look into...

Reactor Netty server and client support trailing headers. The HttpClient exposes the trailing headers via `reactor.netty.http.client.HttpClientResponse#trailerHeaders`, for example: https://github.com/reactor/reactor-netty/blob/89671ea549ce893c74744a602d110cc341a4885e/reactor-netty-http/src/test/java/reactor/netty/http/HttpProtocolsTests.java#L624-L633

@bmaassenee The tests checks for actual trainer header. https://github.com/reactor/reactor-netty/blob/89671ea549ce893c74744a602d110cc341a4885e/reactor-netty-http/src/test/java/reactor/netty/http/HttpProtocolsTests.java#L627 `res.responseHeaders()` are the response headers `res.trailerHeaders()` are the trailer headers

@guptapcam6 Without knowing the exact connection it is hard to say. How do you correlate this message `Message response sent to channel` to some connection?

@guptapcam6 Please use the spring webflux log prefix, it is based on the connection id - https://docs.spring.io/spring-framework/docs/current/javadoc-api/org/springframework/web/server/ServerWebExchange.html#getLogPrefix()

> Curently, boot supports apache, jetty, and okhttp by simply adding those dependencies @spencergibb Is there any reason for not adding Reactor Netty? Some issues?

> So spring-cloud-gateway-server-mvc only depends on spring-boot-starter-web. Users are free to add any third party http client. I cannot understand RestClient supports Reactor Netty out of the box

This one https://github.com/spring-projects/spring-framework/issues/33635 adds auto-detection in Spring Framework for Reactor Netty HttpClient

@FlattestWhite Would you like to create a PR? You didn't mention Reactor Netty version.

> Just wanted to confirm that activeStreams metric isn't available through existing interface. If that's the case, I'll look to create a PR sometime for it. Yeah it is not...