[
https://issues.apache.org/jira/browse/TS-4634?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Leif Hedstrom updated TS-4634:
------------------------------
Fix Version/s: 7.0.0
> HttpCacheSM should not be checking redirection retries
> ------------------------------------------------------
>
> Key: TS-4634
> URL: https://issues.apache.org/jira/browse/TS-4634
> Project: Traffic Server
> Issue Type: Bug
> Components: Cache, HTTP
> Reporter: James Peach
> Fix For: 7.0.0
>
>
> In {{HttpCacheSM.cc}}:
> {code}
> 336 if (open_write_tries > master_sm->redirection_tries &&
> 337 open_write_tries >
> master_sm->t_state.txn_conf->max_cache_open_write_retries) {
> 338 master_sm->handleEvent(CACHE_EVENT_OPEN_WRITE_FAILED, (void
> *)-ECACHE_DOC_BUSY);
> 339 return ACTION_RESULT_DONE;
> 340 }
> {code}
> First, you are not supposed check ``redirection_retries`` without first
> checking ``enable_redirection``. Second, redirection retries should be
> handled above this layer; it seems really hokey to have this check smuggled
> into the HTTP cache layer. Presumably if the HTTP state machine has already
> decided to retry a redirect we should not be second guessing that.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
