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

Sudheer Vinukonda commented on TS-3531:
---------------------------------------

[~degreane] and I debugged the issue he was seeing and it turns out that the 
issue was because his *cacheurl* config file had an empty line and the commit 
b0102b48bcf52a3901fa5373a344f5705350f64a (TS-3345) added stricter config 
validations in 5.3.x. (which is the reason why the same config (with the blank 
line) works fine in 5.2, but, not in 5.3).

We should perhaps enhance the fix in TS-3345 to just ignore blank lines and 
also print a WARNING into diags saying the plugin failed to initialize 
(perhaps, even assert??), if the config load fails due to an error.

Not sure if we should revert TS-3345, since, this may be slightly incompatible 
in the sense that (bad) configs that somehow worked in 5.2 would now fail.

> cacheurl plugin not working in 5.3.0->5.3.x
> -------------------------------------------
>
>                 Key: TS-3531
>                 URL: https://issues.apache.org/jira/browse/TS-3531
>             Project: Traffic Server
>          Issue Type: Bug
>          Components: Cache, Plugins
>            Reporter: Faysal Banna
>             Fix For: 6.0.0
>
>
> Hi Guys 
> was working and doing tests on 5.3.0 before i could integrate it in my 
> production servers, and while doing so i noticed some issues with cacheurl 
> plugin and maybe other plugins but that i can not confirm. 
> Here is what my observation was : 
> in cacheurl.config i have 
> http://s1.2mdn.net/viewad/4079501/BeitMisk_Spring_160x600_web.gif 
> http://tested/switched.gif 
> loaded cacheurl.so cacheurl.config  in plugin.config
> and i confirm that i get [Apr 18 02:45:54.442] Server {0x2b05b69fba80} DIAG: 
> (cacheurl) Adding pattern/replacement pair: 
> 'http://s1.2mdn.net/viewad/4079501/BeitMisk_Spring_160x600_web.gif' -> 
> 'http://tested/switched.gif'
> in traffic.out 
> which says that the pattern was added correctly 
> nevertheless when i do curl or wget with xdebug header request 
> i get : 
> X-Cache-Key: http://s1.2mdn.net/viewad/4079501/BeitMisk_Spring_160x600_web.gif
> meaning that the cacheurl plugin didn't actually do the mapping in any way 
> possible 
> Next was testing with 5.2.1 version of ATS to see how things go so i git 
> cloned it and compiled and ran the same test again 
> now i get 
>  X-Cache-Key: http://tested/switched.gif
> which means that cacheurl in 5.2.1 works while in latest mainstream or 5.3.0  
> it does not 
> i shall do further tests and feed you back here 
> much regards 



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

Reply via email to