Переадресация на несуществующий URI

[alx]

r162/base.

Открыл главную страницу на сервере r2. Хочу посмотреть историю одного из отображаемых там изделий, для этого кликаю серийный номер этого изделия. В результате браузер пытается открыть URL...

http://localhost/index.php?error=1 !!!

И, так как на моем компьютере нет сервера, браузер отображает сообщение об ошибке подключения. Каким образом и почему браузер вдруг пошел на localhost вместо r2?

В исходниках обнаружил два места, где фигурирует localhost (не считая комментариев):

• source:/base/orders.php@162:583#L580
• source:/base/lib/main.lib.php@162:423#L400

В первом из найденных файлов, насколько я понял, при клике какой-то кнопки выполняется такой код:

  window.location.replace("http://localhost/orders.php");

Этот код неправильный сразу по нескольким причинам:

1. Метод replace() типа String требует два аргумента, а не один. Первый - что заменять, второй - на что заменять.
2. В качестве аргумента указан абсолютный URI "​http://localhost/orders.php", при том что в реальной жизни неизвестно, на каком сервере и по какому пути у пользователя будет находиться требуемая страница (и уж наверняка это не будет localhost!). Код должен писаться таким образом, чтобы система была переносимой - то есть она должна работать как в случае, если orders.php находится на http://localhost/orders.php, так и на https://localhost/orders.php, и на https://heaven.org/base/orders.php, и на https://hell.com:666/base/shmaze/my/favotite/long/path/orders.php. То есть работа системы не должна зависеть ни от адреса сервера, ни от протокола, ни от номера порта, ни от пути к корневому каталогу наших файлов.
3. Если я правильно догадался, что идея была в том, чтобы просто перезагрузить страницу, то для этого достаточно было бы сделать window.location = window.location.

Во втором файле сервер внутри функции error403() возвращает HTML-код:

<meta http-equiv="refresh" content="0; url=http://localhost/index.php?error='.$error.'">

Очевидно, это и есть мой случай. :) Возникает сразу ряд вопросов:

• Почему вообще выполнилась error403()? У меня есть право на просмотр истории, и я аутентифицирован в системе (только что перед этим успешно получил список изделий на главной странице). Воспроизвелся баг #878?

• Почему здесь используется абсолютный URI? См. все, что написано выше об абсолютныз URI и переносимости.

• Наконец, выбран довольно странный метод переадресовать браузер на другой URL. Во-первых, <meta/> - это IMHO своеобразный "костыль", который может быть полезен в некоторых (экзотических) случаях. Например для сохранения Content-Type документа при его записи в файл (иначе при последующем чтении HTML-файла браузер не будет знать, как отобразить документ). В нашем же случае документ запрашивается с сервера, и нет никакого смысла перемещать поля из заголовка в meta http-equiv. В-третьих, для немедленной (у refresh стоит период 0!) переадресации протокол HTTP имеет специально для этого предназначенные средства - ответы 301/302/307. Рекомендую пользоваться ими.

[alx]

Да, видимо, воспроизвелся #878 - при попытке снова открыть главную страницу у меня потребовали аутентификации.

[Denis_N]

In 166/base:

Исправлен баг: Переадресация на несуществующий URI

Исправил. Спасибо, Алексей, за пояснение

closes #1083