arturobernalg commented on PR #422: URL: https://github.com/apache/httpcomponents-client/pull/422#issuecomment-1517428956
> > > @arturobernalg Looks good. I can merge this change-set unless you also want to add handling of I/O errors in the same PR. > > > > > > @ok2c Could you please clarify which part of the codebase you are referring to for this change? > > ``` > The stale-if-error HTTP Cache-Control extension allows a cache to > return a stale response when an error -- e.g., a 500 Internal Server > Error, a network segment, or DNS failure -- is encountered, rather > than returning a "hard" error. This improves availability. > ``` > > The change-set introduces `stale-if-error` support for HTTP status 5xx but not for network failures. You do not have to do it. I am going to merge the PR anyway. I am just wondering if you were planning to work on it as well. I see. i'll take a look this afternoon. I'll do the change in another PR TY -- 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]
