arturobernalg commented on PR #609: URL: https://github.com/apache/httpcomponents-client/pull/609#issuecomment-2578317203
> @arturobernalg I must admit it is difficult for me to understand the intent of your changes. You moved some things around and I do not quite understand why. Could you split your changes in two commits, one with refactoring without protocol changes and another one with only protocol changes? @ok2c @garydgregory the purpose of moving the code is to ensure the existing tests pass with the new validation we introduced. Without this adjustment, some of the previous tests would fail due to the added checks for "must-revalidate" and caching responses with Authorization headers. -- 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
