arturobernalg commented on PR #724: URL: https://github.com/apache/httpcomponents-core/pull/724#issuecomment-6312858228
> @arturobernalg And to remedy that you suggest we introduce a super expensive scan over each and every header in one of the hottest execution path of the library? Both classic and async transports preserve CR/LF in a raw FormattedHeader when an explicit UTF-8 encoder is configured. The default path replaces them with ?. Confirmed against clean master. I agree this edge case does not justify scanning every header. The current patch is too broad. -- 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]
