Да я помню как Женю пригласили по части безопастости быть ответственным в MODX Но еще тогда они не прислушивались к его мнению хотя странно зачем приглашать специалиста что б его не слушать 🙂

Женя, расскажи, ты только на безопасности специализируешься?

Да я помню как Женю пригласили по части безопастости быть ответственным в MODX Но еще тогда они не прислушивались к его мнению хотя странно зачем приглашать специалиста что б его не слушать 🙂

Эту договоренность аннулировли через неделю после сотрудничества

Я так понимаю, что Трэш выбрал просто не верный формат партнёрства

Я все прочитал. Но решил озвучить своё мнение.

Причем было это пару недель назад. Вот у тебя выдержка)

Женя, расскажи, ты только на безопасности специализируешься?

EvolutionCMS, сложные проекты или пропетым большими данными на Laravel. Безопасность это скорее хобби (аудиты безопасности modx проектов и не только). Хотя парочку аудитов ICO проектов на ETH сопровождал.

Простите за орфографию. В дороге, а на телефоне автодополнение балуется

C ужасом думаю, что Евгений еще что-то копнет ).

Причем было это пару недель назад. Вот у тебя выдержка)

Мне только сейчас дали ссылки на эти беседы

C ужасом думаю, что Евгений еще что-то копнет ).

Уже есть, но это как обычно воспримется не критично. Поэтому лучше всего фиксит в компонентах

Все тот же getObject()

Что не так с getObject?

Все та же обработка входных параметров

Только на это раз - массивов

Смотрите последний pr в minishop2

Ребят, а то что в мускуле таблицы разных типов для одного сайта лежат, это норма? Какие то InnoDB какое то MyISAM

Ребят, а то что в мускуле таблицы разных типов для одного сайта лежат, это норма? Какие то InnoDB какое то MyISAM

Норма

мускуль там у меня версии 5.7.22

Недавно перенес эту бд на серак клиента, и пришлось подправить дамп под MyISAM все, так как у него на сервере мускуль версии 5.2.22

Женя привет

Поздравляю =)))

👌🏻

Недавно перенес эту бд на серак клиента, и пришлось подправить дамп под MyISAM все, так как у него на сервере мускуль версии 5.2.22

только с версии 5.6.4 MySQL поддерживает полнотекстовые поиск в InnoDB

👌🏻

+

Тоже не сразу заметил

Значит не зря фотик купил, фотки еды стали качественнее и вот теперь вот)

только с версии 5.6.4 MySQL поддерживает полнотекстовые поиск в InnoDB

Они по разному с данными работают ещё. Но это мелочи. Ключевое действительно полнотекстовый поиск

Значит не зря фотик купил, фотки еды стали качественнее и вот теперь вот)

А что лучше всего готовить, чтобы админом стать😂😂

Смотрите последний pr в minishop2

https://github.com/bezumkin/miniShop2/commit/ca47ebbb15b8b13a71b1188423d4136b3a1284ff

пиздишь

чем больше материшься, тем авторитетнее в чате?

А что лучше всего готовить, чтобы админом стать😂😂

еду)))

https://github.com/bezumkin/miniShop2/commit/ca47ebbb15b8b13a71b1188423d4136b3a1284ff

Оно

чем больше материшься, тем авторитетнее в чате?

нет___ Меня просто они бесят

Та же дыра, что и в Tickets. Видимо Василий просто пропустил.

Они по разному с данными работают ещё. Но это мелочи. Ключевое действительно полнотекстовый поиск

Так вот вопрос, могло ли сказаться на работе сайта, в плане тормозов, то что я перенес бд с разными типами на другой хост и все подкрутил под один MyISAM ?

Та же дыра, что и в Tickets. Видимо Василий просто пропустил.

Нет

Обалдеть, как возможно такие простые дыры пропускать

Та, что я описывал пару лет назад пофикшена. И сделано хорошо. Но тут другая тема.

Наверное банальная усталость

Я могу описать все это дело в статье, если интересно. А там пусть сообщество решает степень критичности

Так вот вопрос, могло ли сказаться на работе сайта, в плане тормозов, то что я перенес бд с разными типами на другой хост и все подкрутил под один MyISAM ?

Смотря что за запросы

Обалдеть, как возможно такие простые дыры пропускать

Я думаю тот скрипт был добавлен для примера и больше не трогался.

Смотря что за запросы

Ну банально сайт стал медленее грузиться чем до переноса

Слава богу ни минишоп ни тикетс не использовал.

в pgoTools надеюсь такого нет

pdo*

Слава богу ни минишоп ни тикетс не использовал.

Так это скорее в минус тебе, а не в плюс.

Так это скорее в минус тебе, а не в плюс.

может быть. Мне Шопкипера хватало пока

платить за каждый чих жаба давит

может быть. Мне Шопкипера хватало пока

http://forum.modx-shopkeeper.ru/topic/1059/%D0%B4%D0%B5%D0%BC%D0%BE-%D1%81%D0%B0%D0%B9%D1%82-shopkeeper-4-alpha - четвёртую версию юзаешь?

тем более наумкину

http://forum.modx-shopkeeper.ru/topic/1059/%D0%B4%D0%B5%D0%BC%D0%BE-%D1%81%D0%B0%D0%B9%D1%82-shopkeeper-4-alpha - четвёртую версию юзаешь?

нет еще

платить за каждый чих жаба давит

За который еще чих платить? Тем более Василию? Минишоп бесплатный. У Васи вроде нет ни одного платного компонента для минишопа.

тем более наумкину

Ну у Чирко зато задал вопрос, и можно пару лет ждать пока он ответит, и ещё пару пока пофиксит

Потому я забил вообще работать с шопкипером

ну ещё можно заюзать shopmodxbox)) Но я не знаю, насколько там всё хорошо и развивается ли это)

ну ещё можно заюзать shopmodxbox)) Но я не знаю, насколько там всё хорошо и развивается ли это)

Никак не развивается

Нет

Разве это не тоже самое https://modx.pro/security/10345

ожидаемо кстати)

платить за каждый чих жаба давит

Так пусть клиент платит

Потому я забил вообще работать с шопкипером

там косяков полно, но мне легче пофиксить шопкипер, чем ковыряться в минишоп..

Так пусть клиент платит

пусть он мне платит, а не ему

там косяков полно, но мне легче пофиксить шопкипер, чем ковыряться в минишоп..

Аксиома эскобара так то

Кто знает почему выдает ошибку при загрузке файлов Расширение файла `` не допускается. ??? Хотя в upload_files прописал расширение

Я могу описать все это дело в статье, если интересно. А там пусть сообщество решает степень критичности

Любые статьи на тему безопасности интересны

Нет

А вот исправления в Tickets. Аналогичные твоим. https://github.com/bezumkin/Tickets/commit/b3251a12e557cc7c067a30e97594ba609c59f02d

Если я правильно понимаю, то тот фикс Василия метода xPDO::getCriteria полностью проблему не закрывает.

А все потому, что нужно приучить себя передавать массив в метод вместо просто id. Ну и типы проверять, да

Ребят подскажите стоит ли просить админа обновить мне MySQLна хосте с версии 5.5.59 до 5.7.22, для того чтобы поддерживал полнотекстовые поиск в InnoDB, так как модиск использует и InnoDB и MyISAM в перемешку, я просто не совсем понимаю насколько критично без поддержки InnoDB

Вообще порезать много чего можно, но пользоваться станет не так удобно

Подскажите пожалуйста: в мсИмпортЭкспорт в доке написано "При указании в parent название категории в случае ее отсутствия на сайте она будет автоматически создана, можно также указывать вложенные категории (начиная с базового каталога) разделяя их разделителем который указав в настройках в поле “Дополнительный разделитель для полей типа JSON” (по умолчанию это | ) в таком случае если какой-то из категорий не будет она также будет создана." я указываю в файле импорта категорию в виде "Волосы|Средства для окрашивания и осветления волос" категория "волосы" уже есть на сайте, а вторую надо создать, вроде как всё должно ок быть, но нифига не импортируется и в лог сыпется ошибка Не удалось создать категорию для товара: Array ( [0] => uri: Ресурс с ID 1587 уже использует URI volosyi. Пожалуйста, введите уникальный псевдоним или используйте «Заморозить URI», чтобы вручную заменить его. [1] => alias: Ресурс с ID 1587 уже использует URI volosyi. Пожалуйста, введите уникальный псевдоним или используйте «Заморозить URI», чтобы вручную заменить его. ) Что не так? Почему-то он пытается создать категорию "Волосы", хотя она уже есть.

Ребят подскажите стоит ли просить админа обновить мне MySQLна хосте с версии 5.5.59 до 5.7.22, для того чтобы поддерживал полнотекстовые поиск в InnoDB, так как модиск использует и InnoDB и MyISAM в перемешку, я просто не совсем понимаю насколько критично без поддержки InnoDB

Вот когда разберётесь, тогда и просите. А пока просто работает и смысла нет.

👌🏻

Заслужил.. едой 😁😉

А все потому, что нужно приучить себя передавать массив в метод вместо просто id. Ну и типы проверять, да

Для этого нужно знать имя поля. А оно может не совпадать с id. Проще фильтровать.

Разве это не тоже самое https://modx.pro/security/10345

Можно сказать что это версия 2.0 для моего первого багрепорта getObject, который восприняли как нормальное поведение

А вот исправления в Tickets. Аналогичные твоим. https://github.com/bezumkin/Tickets/commit/b3251a12e557cc7c067a30e97594ba609c59f02d

Это скорее всего фикс после сообщения о баге в личку для @bezumkin

А все потому, что нужно приучить себя передавать массив в метод вместо просто id. Ну и типы проверять, да

Да, ->getObject('xxxx', array('field' => $_GET['field'])) без уязвимости, в отличии от ->getObject('xxx', $_GET['field']);

Да, ->getObject('xxxx', array('field' => $_GET['field'])) без уязвимости, в отличии от ->getObject('xxx', $_GET['field']);

А этот фикс Василия не закрывает проблему? public function getCriteria($className, $type = null, $cacheFlag = true) { $c = $this->newQuery($className); $c->cacheFlag = $cacheFlag; if (!empty($type)) { if ($type instanceof xPDOCriteria) { $c->wrap($type); } elseif (is_scalar($type)) { if ($pk = $this->getPK($className)) { $c->where(array($pk => $type)); } } else { $c->where($type); } } return $c; }

А этот фикс Василия не закрывает проблему? public function getCriteria($className, $type = null, $cacheFlag = true) { $c = $this->newQuery($className); $c->cacheFlag = $cacheFlag; if (!empty($type)) { if ($type instanceof xPDOCriteria) { $c->wrap($type); } elseif (is_scalar($type)) { if ($pk = $this->getPK($className)) { $c->where(array($pk => $type)); } } else { $c->where($type); } } return $c; }

Я думаю понятнее будет если я статью накидаю.

Я за. А Иван, наверно, будет ругаться как обычно 😄