arturobernalg commented on PR #906: URL: https://github.com/apache/httpcomponents-client/pull/906#issuecomment-6625379060
I am not convinced this feature justifies the additional public API and maintenance overhead. It only records known entity lengths, not actual content sizes or bytes transferred. It does not cover compressed or streaming requests, and the original OpenSearch use case is only partially addressed. Applications can already obtain declared lengths through `getContentLength()`. I would prefer not to merge this in its current form. I think we should reconsider whether HttpClient should expose declared entity lengths or actual content size metrics. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
