Project

General

Profile

Actions

Bug #17320

closed

Wrong http-header on Domain-Redirect. Should be 301 instead of 302

Added by Michael Mtzel almost 17 years ago. Updated over 13 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Communication
Target version:
-
Start date:
2007-05-21
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
4.1
PHP Version:
4.3
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

The option "Redirect to:" in the Domain-object should use the header 301 instead of 302.
The patch on "http://bugs.typo3.org/view.php?id=3787" only works with "External link".

(issue imported from #M5660)


Files

class.t3lib_page.php.diff (466 Bytes) class.t3lib_page.php.diff Administrator Admin, 2008-01-14 05:36

Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Feature #16318: Patch: new option TSFE.jumpUrl_prefer301 = 1ClosedJeff Segars2006-07-04

Actions
Actions #1

Updated by Carsten Pohle almost 17 years ago

I just received a mail from Google QM team, telling they will throw us out of the index due to a redirection violating their guidelines - PLEASE raise priority of this issue to CRITICAL or post a workaround!

Actions #2

Updated by Sascha Egerer almost 17 years ago

The Mail from Google is SPAM!
German News about this: http://www.heise.de/newsticker/meldung/89591

Actions #3

Updated by Carsten Pohle almost 17 years ago

In the meantime, I've got a couple more of these mails and already guessed that this is in fact SPAM. However, since the mail contained neither attachments nor any faked URLs, there was no hint that it did not come from Google.

Furthermore, I found some evidence in several webmaster howtos that Google indeed punishes the use of 302 redirects above 301 ones. So I worked around this issue using .htaccess in the meantime.

Sorry for the excitement!

Actions #4

Updated by Ingmar Schlecht over 16 years ago

Hi there,

can some of the persons interested in this bug please send a patch as an RFC to the core list?

Read http://news.typo3.org/news/article/core-development-mailinglist-now-open-for-community-involvement/ and the news for details.

cheers
Ingmar

Actions #5

Updated by Jeff Segars over 16 years ago

The redirect for domain records is controlled in t3lib_page->getDomainStartPage().

I'll get a core patch together for this (all 1 line of it), but should 301 vs 302 be something that is configurable in each domain record? I think my vote would be yes.

Actions #6

Updated by Oliver Klee over 16 years ago

I don't think this should be configurable - we just need to do The Right Thing (TM).

Actions #7

Updated by Jeff Segars over 16 years ago

Attached a one line patch against trunk to fix this. I'll propose it on the core list tomorrow.

Actions #8

Updated by Michael Mtzel over 16 years ago

Yippee! It work. Thanks to Jeff. What is with the 50 Euros?

Actions

Also available in: Atom PDF