htaccess — проблемы с 301 редиректом с GET параметрами, что я делаю не так?

Question

[Иван Волков]

На сайте поменялись урлы и нужно сделать 301 редеректы на старые.
Было my-site.ru/catalog/?branch=3855
Стало my-site.ru/catalog/3855/

Пробовал различные вариации, аля:

RewriteRule ^catalog/?branch=([0-9]+)$ catalog/$1/ [R=301,L]

RewriteCond %{QUERY_STRING} ^[?]?branch=(.*)[&|$] [NC]
RewriteRule /catalog/ my-site.ru/catalog/%1/ [R=301,L]

RewriteCond %{QUERY_STRING} ^branch=([^&]+)
RewriteRule ^.*$ my-site.ru/catalog/%1/ [R=301]

ничего не подошло…

Any Ideas?

Answer 1

[Иван Волков]

Короче, заколебало меня мучатся в итоге и я сделал это на уровне PHP :)

if (strpos($_SERVER['REQUEST_URI'], '/catalog/?branch=') !== false)
{
header(«HTTP/1.1 301 Moved Permanently»);
header(«Location: www.aleshka.by/catalog/».str_replace("/catalog/?branch=","",$_SERVER['REQUEST_URI'])."/");
exit();
}

но всё же, на будущее, хотелось бы знать, как это реализовать на уровне mod rewrite

Answer 2

[dds]

Попробуйте вот такой вариант:

    RewriteCond %{QUERY_STRING} ^branch=([0-9]+)
    RewriteRule ^(.*)$ http://cv.local/$1%1/? [R=301,L]

Answer 3

[ilyautkin]

Попробуйте так:

RewriteCond %{QUERY_STRING} branch=(.*)$ [NC]
RewriteRule ^(.*) /? [R=301,L] 

Comments

[Иван Волков]

Вот оно! Похоже, то что доктор прописал)

RewriteCond %{QUERY_STRING} branch=(.*)$ [NC]
RewriteRule ^(.*) /catalog/%1/? [R=301,L]

так редиректит как надо, на /catalog/id/
но у меня ниже ещё идёт:

RewriteRule ^catalog/(.*)/(.*)$ index.php?page=catalog/&branch=$1&ccp=$2 [QSA] #cpp — paging
RewriteRule ^catalog/(.*)/$ index.php?page=catalog/&branch=$1 [QSA]

если их закоментить, то всё ок, редиректит правильно, но тогда сайт не работает :)
а с ними редиректит на вот такой урл — www.my-site.ru/catalog/3855&ccp=&ccp=&ccp=&ccp=&ccp=&ccp=&ccp=&ccp=&ccp=&ccp=&ccp=&ccp=&ccp=&ccp=&ccp=&ccp=&ccp=&ccp=&ccp=&ccp=/

Что теперь не так?

[pr0tect0r]

у вас получается рекурсия — сначала вы делаете редирект с branch на /catalog, потом опять же catalog реврайтите в branch, вот у сервера и взрывается мозг.

попробуйте перенести условие:

RewriteCond %{QUERY_STRING} branch=(.*)$ [NC]
RewriteRule ^(.*) /catalog/%1/? [R=301,L]
после
RewriteRule ^catalog/(.*)/(.*)$ index.php?page=catalog/&branch=$1&ccp=$2 [QSA] #cpp — paging RewriteRule ^catalog/(.*)/$ index.php?page=catalog/&branch=$1 [QSA]

htaccess последователен, и если в урл не подошел под основное правило, он двигается далее, где происходит редирект, а после уже правила выполняются сначала. И добавьте флаг L в основное правило, говорящий что данное правило последние и не нужно более что-то делать.
RewriteRule ^catalog/(.*)/(.*)$ index.php?page=catalog/&branch=$1&ccp=$2 [QSAL] #cpp — paging RewriteRule ^catalog/(.*)/$ index.php?page=catalog/&branch=$1 [QSAL]

[pr0tect0r]

поправка [QSA,L]

Answer 4

[Владимир Дубровин]

RewriteRule ^/catalog/\?branch=([0-9]+)$ /catalog/$1/ [R=301,L]

Comments

[Иван Волков]

экранировать и ставить слеши перед catalog я пробовал по разному, но потом вычитал что если нужен редирект c GET параметрами, то его нужно делать через RewriteCond + RewriteRule

[Дмитрий Сурин]

Фигню написал. Так это не работает. RewriteRule [Pattern] [Substitution] [Flags]
The Pattern can use a regular expression and is matched against the part of the URL after the hostname and port (with the .htaccess placed in the root dir), !!!but before any query string.!!!

Answer 5

[tegrato]

Вот здесь описаны варианты 301 редиректа с параметрами: https://skalolaskovy.ru/joomla/500-htaccess-301-re...
Задача, правда, немного иная, но принцип можно использовать и вручную прописать все редиректы, без универсальной маски.