arturobernalg commented on PR #430: URL: https://github.com/apache/httpcomponents-core/pull/430#issuecomment-1726330738
> * This change-set is definitely useful, but it kind of goes counter to the overall philosophy of the library. Generally we provide higher level APIs for consumers to use and generate low level HTTP protocol headers based on request details or execution context. Consider our statement management and authentication in client and content transport in core. However if the library consumers manually populate protocol headers we allow them to pass any rubbish they please (with a few notable exceptions) . > > * We presently do not support content rangers at all. I am not sure we ought to mess with headers users populate manually > > * If still want to contribute this request interceptor, I suppose the client module should be a better home for it, not core @ok2c, I appreciate your feedback. I'll investigate integrating this change-set into the client module while respecting the library's philosophy. -- 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: dev-unsubscr...@hc.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@hc.apache.org For additional commands, e-mail: dev-h...@hc.apache.org
