lhotari commented on PR #25211: URL: https://github.com/apache/pulsar/pull/25211#issuecomment-3847551171
> * Added logic in `AdminProxyHandler` to detect requests with the > "Expect: 100-continue" header and wrap them using > `NoExpectRequestWrapper`, ensuring the header is not forwarded to the > broker and preventing Early EOF. The reason why this doesn't make sense is that the client won't be sending a body when the request contains `Expect: 100-continue`. Forwarding such a request to the backend server is not useful. I think it's better to add the continue support using `protocolHandlers.put(new ProxyContinueProtocolHandler())` as mentioned in my previous comment. An alternative approach would be to send a response with status code 100 (continue) without contacting the backend at all. The client would then send the actual request after that. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
