Сообщаем об ошибках phpBB 3.3.x разработчикам

[rxu]

Об ошибках, выявленных в работе phpBB 3.3 Proteus, сообщаем в этой теме.
Ошибками является некорректная работа ядра phpBB, подтвержденная на последней актуальной версии phpBB, требующая устранения в коде ядра, и не связанная с работой расширений, сторонних стилей оформления, неверной работой серверного оборудования или серверного софта, а также с некорректными действиями администратора.
Запросы поддержки также не размещаются в данной теме.

[dimassamid]

dimassamid, а SQL explain что сообщает по этой выборке? Сортировка файлов задейстована?

SQL Report 

mysql explain.png

Отправлено спустя 25 минут 13 секунд:
Установил чистую конференцию и залил туда таблицу со своими юзерами.
rxu, я отправил вам доступ в личку, чтобы можно было посмотреть вживую.

[rxu]

dimassamid, для начала так https://github.com/phpbb/phpbb/pull/6490

[dimassamid]

Благодарю за помощь и интерес к проблеме, но эти правки, к сожалению, её не решили.

[rxu]

С этой правкой одномоментно проблема не уйдет, только по мере посещения конференции пользователями, и потом она не должна возникать, т.к. поле в таблице пользователей будет постоянно актуальным. Для параллельного одномоментной синхронизации понадобится еще миграция, пока не уверен, нужно это, или нет. На крупных конференциях с большим количеством сессий запрос может оказаться тяжелым.

[Татьяна5]

Назначение страницы из расширения главной - костыль есть, решение бы сделать
В $request из /phpbb значение идёт верное, а в request'ах в symphony с чего-то строки до отрабатывания DirectoryIndex из .htaccess. Проверять можно на том же стандартном app.php/help/faq, без правки выходит ошибка "Страница не найдена"

[rxu]

Думаю, что бага ядра тут нет, это пересечение работы вебсервера и системы маршрутизации Symfony. Скорее всего, должно решаться переадресацией в .htaccess с того, что указано в DirectoryIndex (например, с index.php) на app.php/help/faq.

[Татьяна5]

rxu, при переадресации меняется ссылка в адресной строке, так что это не вариант
А просто с DirectoryIndex работает в результате странно. Подхватывает app.php, но не параметры для него (те же /help/faq), и в результате "страница не найдена"

[Nekstati]

У нас в .htaccess есть такие строки:

Код: Выделить всё

RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule ^(.*)$ app.php [QSA,L]

То есть при использовании Apache все 404 запросы идут на app.php. В Nginx, если он настроен по документации PHPBB, тоже.

И если у нас на странице есть хотя бы одна битая ссылка на картинку или другой ресурс, в сессию пользователя вместо адреса страницы записывается чушь в виде app.php/картинка.jpg.

Как воспроизвести: вставить в сообщение битую картинку, например [img]/картинка.jpg[/img], и посмотреть, что записалось в phpbb_sessions.session_page. Ещё лучше вставить картинку в футер, тогда во всех сессиях будет одно и то же.

В результате на viewonline.php местоположение всех пользователей показывается как "Главная страница", а в самой теме, сколько бы пользователей её ни читало, - "Сейчас этот форум просматривают: [текущий пользователь] и 0 гостей".

Как исправить: можно при ошибке 404 не обновлять сессию. Всё равно любая запись туда нерелевантна и даёт всегда "Главную страницу".

Код: Выделить всё

phpbb\event\kernel_exception_subscriber.php
			$message = $this->language->lang('PAGE_NOT_FOUND');
добавить перед (упрощаю):
			$GLOBALS['user']->update_session_page = false;

Или делать там фильтрацию по расширению - не обновлять сессию, если запрошен ресурс типа .jpg .js .css и т.д.

[Siava]

Nekstati, в nginx правильней сделать отдельным location обработку всех картинок и статики, чтобы они выводились напрямую.

[rxu]

тогда во всех сессиях будет одно и то же

А если ссылки на картинки не битые, то всё хорошо, в сессии записываются ссылки на сами страницы?

Отправлено спустя 14 минут 59 секунд:
Посмотрел, видимо, слушатель исключения срабатывает уже после session_begin(), так что простой способ типа $GLOBALS['user']->update_session_page = false не сработает.

[Nekstati]

Да, если ссылка не битая, сервер отдаёт картинку напрямую, а если битая, то перенаправляет на app.php, и далее app.php перезаписывает то, что viewtopic.php записал в сессию.

Способ работает, по крайней мене на моей локалке. Я никогда не пощу код, не проверив его.

Для вложений, например, давно предусмотрено update_session_page = false в виде $user->session_begin(false);

[rxu]

https://tracker.phpbb.com/browse/PHPBB3-17286

[romaamor]

Как исправить

Так как исправить то ?

[rxu]

Хак с $GLOBALS['user']->update_session_page точно не пойдет в ядро.

Отправлено спустя 1 час 12 минут 34 секунды:
Вот так должно сработать: https://github.com/phpbb/phpbb/pull/6582

[romaamor]

Вот так должно сработать:

А по простому как ? Там не чего не понятно что и куда.