Download & Extend

G2 should be able to use 301 redirects

Project:G2 Glossary
Version:6.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:fgm
Status:closed (fixed)

Issue Summary

The current version of G2 uses Temporary Redirects (HTTP 302) to automatically link to a node page when only one homonym is found and the "Enable auto-jump to single match" feature is enabled on the admin/settings/g2 page. It is enabled by default.

Since these redirects are built from a database query over a varying set of definitions, they are indeed temporary. However, due to the HTTP 302 search engine exploit, some webmasters might prefer to use Permanent Redirects (HTTP 301) for fear of search engine blacklisting.

Comments

#1

G2 uses the builtin drupal_goto() function, which doesn't do 301 (yet ?) .

See http://drupal.org/node/75803

#2

Version:4.7.x-1.x-dev» master

Submitting patch.

AttachmentSizeStatusTest resultOperations
common.inc_30.patch1.18 KBIgnored: Check issue status.NoneNone

#3

Assigned to:fgm» Anonymous
Status:active» needs review

Version 1.15.2.8 adds manual support for this behaviour, until http://drupal.org/node/75803 is committed.

Look in the code for $redirect_type (near line 2142).

#4

Status:needs review» postponed

The required core patch has been applied to Drupal 5.0, but the backport to 4.7 has been rejected, so this update is postponed until a 5.0 version of G2 is created.

#5

Version:master» 4.7.x-1.x-dev

Although the needed core patch (http://drupal.org/node/75803) has be requested again for other needs, it has been rejected again, so the G2 patch stays postponed until a 5.0 version of G2 is created.

#6

Version:4.7.x-1.x-dev» master
Status:postponed» active

Drupal 5.0 and 6 now support this.

#7

Version:master» 6.x-1.x-dev
Assigned to:Anonymous» fgm

Switching to the current dev branch

#8

Status:active» fixed

Included in today's version v1.19.2.2.

The code had actually been there for ages, but commented out due to http://drupal.org/node/75803 rejection for 4.7.

#9

Status:fixed» closed (fixed)