Closed (fixed)
Project:
Google AdSense integration
Version:
7.x-1.0
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
17 Jan 2013 at 11:02 UTC
Updated:
17 Jul 2013 at 10:21 UTC
Hello,
first i think this is outdated while google ads supports https.
I think that in each case at least a placeholder with a help-text should be displayed!
After 2h of trying to configure without success, i foud this "ugly" code:
File: adsense.module
in function _adsense_page_match()
...
if (isset($_SERVER['SERVER_SOFTWARE']) && (stristr($_SERVER['SERVER_SOFTWARE'], 'Apache') !== FALSE) &&
isset($_SERVER['HTTPS']) && ($_SERVER['HTTPS'] == 'on')) {
return FALSE;
}
greetings!
Comments
Comment #1
jcnventuraNo they don't: https://support.google.com/adsense/bin/answer.py?hl=en&answer=10528
Removing those lines will make the browser warn users about unsecure elements in the page, which is very annoying. But you're right, there should be an option to allow you to annoy your users.
Comment #2
jcnventurahttp://drupalcode.org/project/adsense.git/commit/718e9c2
Feel free to annoy your users.