Active
Project:
GeoIP Redirect
Version:
6.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Reporter:
Created:
12 Apr 2011 at 10:01 UTC
Updated:
13 Feb 2012 at 17:59 UTC
great module, but I didn't like the popup and the delay in redirection, can't we just redirect it instantly with no popup so the user don't know about being redirected as that is the main purpose of the module, so how to do it ? :)
Comments
Comment #1
btopro commentedActually it was originally done that way and had to be changed. What you described conflicts with a lot of search engine policies (namely Google). You can't automatically redirect traffic from one link to another. While it seems like that's what you want to do this can get you Blacklisted from Google and other search engines because this is seen as nefarious activity. I think the reasoning is that if I'm in the UK on a free and open internet there should be nothing stopping me from going to your US site (as an example). The popup message gives them the option to leave the site and go to the redirected one in question. If they say yes and want the choice remembered it will then automatically redirect but you can't force them to redirect.
Comment #2
rmcom commentedI would be glad to work on a patch to address the three issues that are currently open, so that we could get to v6.x-1.0.
You said:
It would be nice to have this as an option for the site admin to decide on the module settings page.
Would you mind posting the code that you originally had for the autmatic redirect?
Comment #3
btopro commentedI would need a really strong reason as to why to add this in. In pursuit this functionality the site I deployed it on was blacklisted from google for a few days as this technique violates their tos. I'd be all for patches to this or adding on as a maintainer but would need solid argument for this and lots of documentation about potential ramifications.
Comment #4
rmcom commentedI would like to make the fix for the issue I posted myself (http://drupal.org/node/1333524):
- admin page link
- multiple redirects
As for the automatic redirect:
I am surprised this is such a big issue. Google does this itself: when you type google.com in France, it will automatically take you to google.fr based on your ip address. Then, on google.fr, you have a link to google.com that allows you to go and stay there if that's really what you wanted.
I would say that nothing would be wrong with adding this as an option, that the site admin can switch to, but we could add a warning text next to the switch indicating that search engines may blacklist you in some cases.
What do you think?
Comment #5
btopro commentedI'm cool with that. This was really a 1off, if your like to be a maintainer to see this through to a stable release let me know. Otherwise people could use the patch you build here. If you want to combination just open another issue requesting
Comment #6
rmcom commentedCool, I'll start working on the patch for http://drupal.org/node/1333524.
Please copy the code for the automatic redirect here or send it to me through a drupal.org user message.
Once the patch is here, you decide how to proceed.
Thanks!
Comment #7
btopro commentedSounds good. Unfortunately I don't have code anymore because it was super early in implementation and this was contributed after it was finished for a client.
To point in the right direction you have to place the code in hook_boot to avoid page caching issues that would block the redirect from happening. Also this is incompatible with aggressive caching for that reason.