[ 
https://issues.apache.org/jira/browse/TS-4396?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15360139#comment-15360139
 ] 

ASF GitHub Bot commented on TS-4396:
------------------------------------

GitHub user mingzym opened a pull request:

    https://github.com/apache/trafficserver/pull/786

    TS-4396: fix number_of_redirections off-by-one

    please test

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/mingzym/trafficserver TS-4396

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/trafficserver/pull/786.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 #786
    
----
commit 1f0a4635647e2b219ca7a97185545be0bf556165
Author: Zhao Yongming <[email protected]>
Date:   2016-07-02T11:09:08Z

    TS-4396: fix number_of_redirections off-by-one

----


> Off-by-one error in max redirects with redirection enabled
> ----------------------------------------------------------
>
>                 Key: TS-4396
>                 URL: https://issues.apache.org/jira/browse/TS-4396
>             Project: Traffic Server
>          Issue Type: Bug
>          Components: Core, Network
>            Reporter: Felix Buenemann
>            Assignee: Zhao Yongming
>             Fix For: 7.0.0
>
>
> There is a problem in the current stable version 6.1.1 where the setting 
> proxy.config.http.number_of_redirections = 1 is incorrectly checked when 
> following origin redirects by setting proxy.config.http.redirection_enabled = 
> 1.
> If the requested URL is not already cached, ATS returns the redirect response 
> to the client instead of storing the target into the cache and returning it 
> to the client.
> The problem can be fixed by using proxy.config.http.number_of_redirections = 
> 2, but we are only following one redirect, so this is wrong.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to