I'm still not sure whether the correct behaviour is to send ICP for the rewritten URL, or to rewrite the URLs being received before they're looked up. Hm! Adrian 2008/12/24 Imri Zvik <imriz@xxxxxxxxxx>: > On Wednesday 24 December 2008 17:01:39 Adrian Chadd wrote: >> Thanks. Be sure to comment on the bugzilla ticket too. >> >> Oh and tell me which bug it is so I can make sure I'm watching it. :) >> >> >> Adrian >> >> 2008/12/23 Imri Zvik <imriz@xxxxxxxxxx>: >> > On Sunday 21 December 2008 10:52:42 Imri Zvik wrote: >> >> Hi, >> >> >> >> On Thursday 18 December 2008 21:57:22 Adrian Chadd wrote: >> >> > Nope, I don't think the storeurl-rewriter stuff was ever integrated >> >> > into ICP. >> >> > >> >> > I think someone posted a patch to the squid bugzilla to implement >> >> > this. >> >> >> >> If you can point me to said patch, I'd be happy to test it under load. >> >> >> >> > I'm happy to commit whatever people sensibly code up and deploy. :) >> >> > >> >> > >> >> > >> >> > Adrian >> >> > >> >> > 2008/12/18 Imri Zvik <imriz@xxxxxxxxxx>: >> >> > > Hi, >> >> > > >> >> > > I'm using the storeurl_rewrite feature to store content with >> >> > > changing attributes. >> >> > > >> >> > > As my traffic grows, I want to be able to add cache_peers to share >> >> > > the load. >> >> > > >> >> > > After configuring the peers, I've found out that all my ICP queries >> >> > > results with misses. >> >> > > It seems like the storeurl_rewrite logic is not implemented in the >> >> > > ICP queries - i.e., nor the ICP client or the server passes the URL >> >> > > through the storeurl_rewrite process before checking if the >> >> > > requested content is cached or not. >> >> > > >> >> > > Am I missing something? >> >> > > >> >> > > >> >> > > >> >> > > Thank you in advance, >> >> >> >> Thanks! >> > >> > I've found the said patch in squid's bugzilla - It seems to be working, >> > but I'm going to test the patch under load (700 mbit~) and report back. > > Here is the bug report: http://www.squid-cache.org/bugs/show_bug.cgi?id=2354 > > The patch works flawlessly so far. > >
