Сергей Шлоков

Сергей Шлоков

С нами с 31 января 2013; Место в рейтинге пользователей: #3
Сергей Шлоков
15 ноября 2016, 08:02
+2
Запрет возможности подмены пути при запуске $modx->runProcessor [#13176]
Запрет неавторизованного доступа к процессорам [#13175]
Запрет подмены пути в параметре action объекта modConnectorResponse [#13173]
Может через пару месяцев напишешь про то, как это можно было сделать? Интересно. Ведь даже авторы MODX не сообразили.
Сергей Шлоков
15 ноября 2016, 07:57
+11
Справедливости ради, кабы Николай не поднял эту волну, так и сидели бы ровно за «стеклянной дверью». Да и тебе профит вон какой прилетел. :) И ребят из MODX попинали. 2 года назад ты один их не смог растолкать. А щас толпой навалились и результат почти мгновенный. Даже Марк по-русски заговорил :)) Прогресс на лицо. )
Сергей Шлоков
14 ноября 2016, 15:56
+5
Дядя Женя, простишь их? :)
Сергей Шлоков
14 ноября 2016, 07:15
0
It's great!
I know the man who knows some vulnerabilities of MODX. But he got a not very good experience in communicating with the MODX team. And the MODX community suffer from this misunderstanding between you and him.
Сергей Шлоков
14 ноября 2016, 06:59
0
Ещё пропала информация о закачках в блоке информации о компонентах, упоминающихся в статье.
Сергей Шлоков
13 ноября 2016, 19:41
+3
Тема явно произвела впечатление.
Сергей Шлоков
13 ноября 2016, 13:21
0
Иииииихаааааааа. )) Теперь держимся крепче, чтобы не вывалиться.
Сергей Шлоков
13 ноября 2016, 08:57
+6
Видимо они вовсю пользуются возможностью передавать сырой SQL. Им твоя заплатка поломает сайты.
А на своих сайтах нам надо повесить предупреждение для хакеров, чтобы они не посылали чистые sql запросы, ведь нужно использовать первичные ключи. Так Джейсон сказал!!!
Немного конспирологии… А может они специально оставляют эти дырки по просьбе АНБ. :)
Сергей Шлоков
12 ноября 2016, 12:45
0
Точно. Вернул всё взад, теперь работает как надо.
Сергей Шлоков
12 ноября 2016, 12:37
0
В общем, у меня на сайте, если открыть любую статью получаю количество просмотров для первой статьи. Данные формирует сниппет TicketMeta.
Сергей Шлоков
12 ноября 2016, 12:09
+1
Василий, а так должно быть?
После обновления не мог понять, почему у меня при просмотре любой статьи показывает одни и те же данные о просмотре (только первой).
Сергей Шлоков
12 ноября 2016, 10:46
0
Я себе добавил. Заодно и потестирую.
Сергей Шлоков
12 ноября 2016, 10:42
0
Предложил код автору xPDO, буду ждать ответа.
Не забывай, сейчас выходные. Да ещё Рождество и Новый год скоро… Надеюсь, хотя бы в третьей версии MODX добавят. Осталось набраться терпения. :)
Ибо неведомы автору xPDO наши переживания, чего ему ноги ломать…
Сергей Шлоков
12 ноября 2016, 08:52
+1
Но я уже ни в чём не уверен.
Я тоже. В этом плане знаний маловато. Просто исхожу из комментария Евгения
Это проблему не решает. Есть куча точек в хода из различных компонентов, которые оставляют коннекторы в /assets/components/
Насколько я понимаю, именно для этого скрывается папка с коннекторами, а тут лежит в открытом доступе.
Очень хочется, чтобы Евгений успокоил насчет этого.
Сергей Шлоков
12 ноября 2016, 08:22
+2
Думаю, так правильней.
Посмотрел 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 должен в процессоре смотреть мету ключа и фильтровать.
Сергей Шлоков
12 ноября 2016, 07:33
0
А сегодня весь день буду обновлять свои дополнения на предмет фильтрации этих слепых SQL в запросах.
Нам всем теперь нужно это сделать. Было бы неплохо написать про это инструкцию.
Сергей Шлоков
11 ноября 2016, 22:15
0
Это не ко мне. Это как если бы спросил про влияние планеты Нептун на рождаемость сурикатов в неволе.
Сергей Шлоков
11 ноября 2016, 21:38
0
Этот тег отвечает за относительные адреса в ссылках, путях картинок и т.п. Он не влияет на адрес внутренних страниц. Видимо где-то сломана переадресация.
Сергей Шлоков
11 ноября 2016, 21:36
+1
Невозможно перейти на по этой ссылке. Видимо Василий зафильтровал всё нафиг. :)