Комментарии / Сергей Шлоков / Пользователи / Русскоязычное сообщество MODX

Василий, а так должно быть?
После обновления не мог понять, почему у меня при просмотре любой статьи показывает одни и те же данные о просмотре (только первой).

Новые дополнения и их версии [Tickets] 1.7.0 улучшения рейтингов и ускорение работы 55

12 ноября 2016, 10:46

Я себе добавил. Заодно и потестирую.

Безопасность Критическая уязвимость в MODX Revolution 340

12 ноября 2016, 10:42

Предложил код автору xPDO, буду ждать ответа.

Не забывай, сейчас выходные. Да ещё Рождество и Новый год скоро… Надеюсь, хотя бы в третьей версии MODX добавят. Осталось набраться терпения. :)
Ибо неведомы автору xPDO наши переживания, чего ему ноги ломать…

Безопасность Критическая уязвимость в MODX Revolution 340

12 ноября 2016, 08:52

Но я уже ни в чём не уверен.

Я тоже. В этом плане знаний маловато. Просто исхожу из комментария Евгения

Это проблему не решает. Есть куча точек в хода из различных компонентов, которые оставляют коннекторы в /assets/components/

Насколько я понимаю, именно для этого скрывается папка с коннекторами, а тут лежит в открытом доступе.
Очень хочется, чтобы Евгений успокоил насчет этого.

Безопасность Критическая уязвимость в MODX Revolution 340

12 ноября 2016, 08:22

Думаю, так правильней.
Посмотрел Tickets (только он открыт). mSearch2 и Office обновил не глядя. Данный фикс немного меня успокоил. Я всегда так фильтрую данные в своих компонентах (на всякий ещё проверю все). Я про это даже статью год назад написал. Возможно Евгений её раскритикует, но для обсуждаемого случая она актуальна.
Но этот фикс решает половину проблемы. Что делать с коннектором (connector.php)? Евгений заходит через него. Запрос в ObjectGetProcessor с кривым id и милости просим. MODX это не фильтрует.
Т.е. надо переопределять метод initialize для всех get/update/delete процессоров и фильтровать в нём id.

public function initialize() {
        $primaryKey = (int) $this->getProperty($this->primaryKeyField,false);
	$this->setProperty($this->primaryKeyField, $primaryKey)
	return parent::initialize();
}

Очень хотелось бы услышать мнение Евгения по этому поводу.
П.С. По хорошему, MODX должен в процессоре смотреть мету ключа и фильтровать.

Безопасность Критическая уязвимость в MODX Revolution 340