Skip to content

Conversation

ldaley
Copy link
Contributor

@ldaley ldaley commented Jun 9, 2021

Fixes #14433

Redirection is always supported, with up to 10 redirects per request.

A new useExpectContinue configuration option has been added that enables use of HTTP expect-continue with store requests, for environments where store requests are routinely redirected and doubly transmitting the cache entry payload is undesirable.

@ldaley ldaley added this to the 7.2 RC1 milestone Jun 9, 2021
@ldaley ldaley self-assigned this Jun 9, 2021
@ldaley ldaley requested a review from a team June 9, 2021 05:45
@ldaley
Copy link
Contributor Author

ldaley commented Jun 9, 2021

@lptr this is ready for an initial review.

At least, the user docs and other documentation will need to be updated. I'll wait until the functionality and API is approved before doing that in another round.

@gradle gradle deleted a comment from ldaley Jun 9, 2021
@gradle gradle deleted a comment from ldaley Jun 10, 2021
Copy link
Member

@lptr lptr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I left a few minor comments.

@ldaley ldaley requested a review from pioterj as a code owner June 17, 2021 07:06
@ldaley ldaley requested a review from lptr June 17, 2021 07:06