Jira opened for this work. https://issues.apache.org/jira/browse/TS-4017
"Objective here is to allow proxy.config.http.redirection_enabled override to be set in all transaction hooks up to and including 'READ_RESPONSE_HDR'" On Wed, Nov 11, 2015 at 5:49 PM, Jeremy Payne <[email protected]> wrote: > Thanks James that would be ideal at least in my case. > > As a fyi, and for historical purposes I was able to override redirect > within TXN_START. However, doesn't seem like I have access to request > information to make certain decisions. I will run this again with debug > enabled to see whats available during this hook point. > > -- START LUA SCRIPT > function do_global_txn_start() > ts.http.config_int_set(TS_LUA_CONFIG_HTTP_ENABLE_REDIRECTION, 1) > return 0 > end > -- END LUA SCRIPT > > > On Wed, Nov 11, 2015 at 4:56 PM, James Peach <[email protected]> wrote: > >> >> > On Nov 11, 2015, at 12:26 PM, Sudheer Vinukonda <[email protected]> >> wrote: >> > >> > Which hook are you using to override? >> > >> > The config is applied before READ_REQUEST_HDR hook, so, you will need >> to override it in TXN_START hook. >> >> That seems unfortunate. Can we move that back to READ_RESPONSE_HDR? >> >> > >> > Thanks, >> > >> > Sudheer >> > >> > >> > >> > On Wednesday, November 11, 2015 10:12 AM, Jeremy Payne < >> [email protected]> wrote: >> > >> > >> > >> > >> > I see that master(6.1.0) now 'supports' redirection override. >> > >> > ./trafficserver/proxy/http/HttpConfig.cc: >> HttpEstablishStaticConfigByte(c.oride.redirection_enabled, >> "proxy.config.http.redirection_enabled"); >> > ./trafficserver/proxy/http/HttpConfig.cc: >> params->oride.redirection_enabled = >> INT_TO_BOOL(m_master.oride.redirection_enabled); >> > >> > >> > In testing I see that the redirection value is changed per the >> over-ride config(witnessed via debug output), yet I dont see the override >> value is applied to the session/transaction. >> > >> > Is the override code fully implemented? As it seems only the override >> code is there but maybe the transaction code hasnt been updated to account >> for the redirection override. >> > >> > >> > Thanks! >> > >> > >> > >> > >> > >> >> >
