Size: a a a

2020 September 10

СП

Сергей Предводителев... in Yii Framework 3
Alexander Makarov
Скорее всего это значит что он не смог геренуться где-то. То есть произошёл фейл, с которым у тебя не будет работать весь POST. Я бы кидал исключение.
Пользователь увидит ошибку, даже не увидев формы собственно.

Я склоняюсь к варианту  вывести пустой мета-тег <meta name="csrf"> .
Так у пользователя будет после отправки формы ошибка. А программист сможет увидеть, что у него csrf пустой в коде.

Но минус, что в логи это никуда не пойдёт, так как не будет исключений...
источник

AM

Alexander Makarov in Yii Framework 3
Сергей Предводителев
Пользователь увидит ошибку, даже не увидев формы собственно.

Я склоняюсь к варианту  вывести пустой мета-тег <meta name="csrf"> .
Так у пользователя будет после отправки формы ошибка. А программист сможет увидеть, что у него csrf пустой в коде.

Но минус, что в логи это никуда не пойдёт, так как не будет исключений...
А почему может CSRF не быть?
источник

AM

Alexander Makarov in Yii Framework 3
Это же мы его генерим...
источник

СП

Сергей Предводителев... in Yii Framework 3
Alexander Makarov
А почему может CSRF не быть?
мидлварь не подключил например :)
источник

AM

Alexander Makarov in Yii Framework 3
Сергей Предводителев
мидлварь не подключил например :)
Ну, это фейл конфига. Лучше его выловить локально и сразу.
источник

СП

Сергей Предводителев... in Yii Framework 3
Alexander Makarov
Ну, это фейл конфига. Лучше его выловить локально и сразу.
оке, сделаю исключение
источник

NO

Nex Otaku in Yii Framework 3
Лучше если он введëт 100 полей и потеряет всë?) Нет, чем раньше выявляется поломка, тем лучше. Исключение, норм вариант.
источник

AM

Alexander Makarov in Yii Framework 3
Nex Otaku
Для меня, это показатель неумения проектировать. Когда вместо того, чтобы работать со сложностью, еë пытаются просто куда-то упрятать.

Некоторые, например, фигачат события и обработчики, постепенно превращая проект в событийного спагетти-монстра, который невозможно тестировать, который глючит при любом изменении.

Кто-то плодит иерархии классов и злоупотребляет наследованием, пытаясь в родительских классах спрятать всë...

Вот теперь ещë вариант, когда "сделаем контроллеры одинаковыми, а логику спрячем в конфиг".

В чëм смысл контроллера тогда? Если он везде одинаковый, то он и не нужен, все контроллеры можно одной мидлварью заменить)
Ну плохой пример просто был с контроллером :)
источник

AM

Alexander Makarov in Yii Framework 3
Речь чаще про сервисы.
источник

AM

Alexander Makarov in Yii Framework 3
Например, в консольке у нас по дефолту долгие операции - норм и таймауты на коннекшн не нужны. А в API публичном — это зло и надо таймаут. Получается два разных конфига для одного типа сервиса с одним интерфейсом Connection.
источник

T🐜

The Ant 🐜 in Yii Framework 3
Nex Otaku
Для меня, это показатель неумения проектировать. Когда вместо того, чтобы работать со сложностью, еë пытаются просто куда-то упрятать.

Некоторые, например, фигачат события и обработчики, постепенно превращая проект в событийного спагетти-монстра, который невозможно тестировать, который глючит при любом изменении.

Кто-то плодит иерархии классов и злоупотребляет наследованием, пытаясь в родительских классах спрятать всë...

Вот теперь ещë вариант, когда "сделаем контроллеры одинаковыми, а логику спрячем в конфиг".

В чëм смысл контроллера тогда? Если он везде одинаковый, то он и не нужен, все контроллеры можно одной мидлварью заменить)
контроллер нужен чтобы сказать кому-то что делать с входными данными, и проконтролировать процесс :D
Ничо лишнего там быть не должно.
источник

NO

Nex Otaku in Yii Framework 3
The Ant 🐜
контроллер нужен чтобы сказать кому-то что делать с входными данными, и проконтролировать процесс :D
Ничо лишнего там быть не должно.
ну так и контроллер можно конфигом заменить?
источник

AM

Alexander Makarov in Yii Framework 3
Чёт вы оба не в ту степь :)
источник

AM

Alexander Makarov in Yii Framework 3
Хватит развивать нежизнеспособный пример :)
источник

AB

Alexander Borisov in Yii Framework 3
Alexander Makarov
Минус текущего решения несколько надуманный. Есть common. Если сервис общий, он там описывается.
минус видится в большом количестве конфигов, которое плодит текущее решение. если б их было меньше, то, возможно их мердж не был бы такой тяжелой операцией. ну и вообще упростил бы в целом приложение.
источник

NO

Nex Otaku in Yii Framework 3
Alexander Makarov
Например, в консольке у нас по дефолту долгие операции - норм и таймауты на коннекшн не нужны. А в API публичном — это зло и надо таймаут. Получается два разных конфига для одного типа сервиса с одним интерфейсом Connection.
Это как раз второстепенные вещи, в твоëм примере. Есть таймаут, нет таймаута. Это и должно быть в конфиге.
источник

AB

Alexander Borisov in Yii Framework 3
Alexander Makarov
Например, в консольке у нас по дефолту долгие операции - норм и таймауты на коннекшн не нужны. А в API публичном — это зло и надо таймаут. Получается два разных конфига для одного типа сервиса с одним интерфейсом Connection.
ну это можно решить и в рамках одного контейнера. в текущем варианте - два конфига, в вариент с одним конфигом - просто подсовываем в нужное место другую реализацию (если будут какие-нибудь локальные биндинги, то вообще класс)
источник

AM

Alexander Makarov in Yii Framework 3
ℹ️ @predvoditelev согласился влиться в нашу группу разработчиков. Дал прав.
источник

А

Алексей R in Yii Framework 3
источник

СП

Сергей Предводителев... in Yii Framework 3
Hey there, we’re just writing to let you know that you’ve automatically started watching several repositories on GitHub.
источник