[Plugin: Ad Injection] Cookie prevents server-side caching

Hi moepstar, the cookie is for the ‘search engine visitors only’ feature. If a user visits from a search engine then the cookie is set so that ads are shown on subsequent pages.

There is also a cookie relating to blocking adverts.

Are you using the ‘search engine visitors only’ feature?

For the ‘search engine visitors only’ feature the cookie should (in theory) only be used if the box is ticked, but looking at the code it seems to set it regardless.

If you aren’t using this feature you could try changing the condition at the bottom of adshow_show_adverts() in adshow.php to the below. I haven’t tested this but let me know what happens if you try it.

// Only set the cookie if we search engine only option is ticked
if (adinj_config_allow_referrers()){
	//Set cookie
	if (!headers_sent()){
		setcookie('adinj', '1', time()+3600, '/');
	}
}

No, the “search engine only” option is disabled but it sets the adinj cookie regardless.

I haven’t had the time yet to test the proposed fix yet but will report back.. Maybe it’s time to roll out a new release? ??

Hi moepstar, If the proposed fix works for you then I can get it released. Do let me know if it helps.

Ok, that seems to have worked – no more cookies are set and the ads are being displayed…

Setting this to resolved and hope it makes it into the next release!

Hi moepstar, thanks for trying it. I’ll test the fix in more detail and if it is ok it will be in the next release.