GitHub user shinrich opened a pull request:
https://github.com/apache/trafficserver/pull/215
Ts 3656
Changes to allow activating follow redirection in send server response
hook. Originally this would not work for the post case, because the buffers
storing the post data for redirect were being freed too early. Delayed the
cleanup until later. Also changed one of the buffers stored with the post
state to be set up as local variables instead.
Also since the redirection decision is made late, we needed to make a copy
(reference counted copy) of the post buffer regardless what the
enable_redirection flag value happens to be at the time the post data is
received from the client.
Given that the redirect and post logic are both squirrelly, I'd appreciate
another set of eyes looking this over before I commit.
I have validation from a production environment that this patch works to
enable the functionality.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/shinrich/trafficserver ts-3656
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/trafficserver/pull/215.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #215
----
commit 137ef1d7b98566b80335a28987832d9d11413b90
Author: shinrich <[email protected]>
Date: 2015-06-03T18:35:00Z
Enable activating follow redirect later on the transaction lifecycle.
commit 4de9a99df9752060c30e71b7b33f977a14e196e6
Author: shinrich <[email protected]>
Date: 2015-06-09T14:50:30Z
clang format
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
