Павел
Ну короче какждый хочет так и делает, тут критикала ноль
The Ant
ну да, 2 интерфейса, 2 реализации и 1 аутентификатор. Нахрен бандл не понимаю
Vlad
Вот что мне не нравится в лексике, так это то что деграеться каждый раз ключи
Vlad
ну то есть в чем проблема сразу заинжектить их
The Ant
Потом рефреш еще напиши
это 1 контроллер )
Павел
это 1 контроллер )
Это сущность, это репа, это все вот это)
Vlad
да это можно поправить самому
https://github.com/lexik/LexikJWTAuthenticationBundle/blob/2.x/Services/KeyLoader/AbstractKeyLoader.php#L107
The Ant
Это сущность, это репа, это все вот это)
это все ложится в 1 модуль ) опять же, такой, какой тебе надо. Можешь добавить в бд юзерагенты, ипы всякие. И потом это обрабатывать как-то.
Павел
это все ложится в 1 модуль ) опять же, такой, какой тебе надо. Можешь добавить в бд юзерагенты, ипы всякие. И потом это обрабатывать как-то.
Могу и не на симфе писать. Могу доктрину не юзать. Могу валидатор не юзать, могу DI не юзать. Весь вопрос в профите. Пока ты профита не назвал, кроме "потому что я могу сам"
Vlad
его профит не тащим вендор
The Ant
профит в том, что тебе не надо следить за этой зависимостью
Vlad
такой себе профит
The Ant
для тебя может это и не профит совсем конечно
Vlad
вроде не мобильное приложение пишем)
Павел
Я ее использую в 10 строчках модификации
The Ant
за тем что в нее могут напихать всякого )
The Ant
а ты и не заметишь
Павел
за тем что в нее могут напихать всякого )
Без обновления должно быть ок.
Vlad
секурити чтоли?
Павел
за тем что в нее могут напихать всякого )
Во всё могут напихать всякого, это уже параноя
The Ant
Без обновления должно быть ок.
как жить без обновлений? :D
Юра
Проблема в самописе в том что никто кроме тебя не шарит в этом самописе
Юра
Новый проект, стоит jwt bundle, сразу понимаешь что куда
Павел
как жить без обновлений? :D
С утра ты мне нравился больше ))))))))
The Ant
Во всё могут напихать всякого, это уже параноя
были же прецеденты, почему думаешь что больше не будет? )
Vlad
Во всё могут напихать всякого, это уже параноя
пускай в сисписи пихает проверки на всякое
Vlad
тулзы вроде есть
Павел
были же прецеденты, почему думаешь что больше не будет? )
Они были, но можно не обновлять если параноя. Или себе зафоркать
The Ant
или свое юзать )
Павел
или свое юзать )
Не юзай фрейм
The Ant
и исключить лишний гемор ))
Павел
Все просто
Vlad
и не юзать пхп
Vlad
мало ли
Vlad
что то подсунут
The Ant
вот да, пхп тоже ломали ))
Юра
Свое обычно полная хрень, в плане никто на свое не пишет доку, тесты, потом разраб ушел и проект превратился в кучу самописа без доки, считай проект помер
Юра
А если ты написал доку и тесты, то можно и поделиться с миром, но в мире уже есть джвт бандл
The Ant
что то подсунут
пропустил наверное бугурт, когда с подписью попова эксплойт в пхп засунули )
The Ant
🤣🤣
Юра
В нем будут только твои эксплоиты )
Павел
Всем привет. Сейчас вроде симфа предлагает такие методы репо в мейкере. public function save(User $entity, bool $flush = false): void { $this->getEntityManager()->persist($entity); if ($flush) { $this->getEntityManager()->flush(); } } Мне кажется, или этот bool какой то треш?
Shokha
вы можете сразу сохранить после персиста или оставить это на потом
Юра
Вообще треш. Как по мне репозиторий не должен флашить
Null
А в умных книжках говорят, что буль флаги в методах - говно
Павел
вы можете сразу сохранить после персиста или оставить это на потом
То что он делает, я понимаю. Это архитектурный треш, я про это. 1.bool - в аргементах всегда говорит о разбитие на 2 метода 2. Контракт репозитория, имеет какой то flush (представим что мы не знаем что это UoF а видим только контракт), что это вообще такое. Save и какой то другой save который не save, но в тоже время save
Юра
Это слишком важная фция чтобы она где-то там внутрях вызвалась сама собой
Юра
А еще круче когда внутри еще и где-то clear вызывают
Павел
Вообще треш. Как по мне репозиторий не должен флашить
Ну это отдельный холиварчик, не буду его поднимать. Например если у нас не uow то репо будет сохранять, по другому никак
Юра
А ты сидишь три часа и дебажишь
Павел
Ну либо отдельный сервис
Павел
Я бы еще понял, если бы внутри был тупо всегда персист и флаш, но в таком виде - не понимаю.
Юра
Это все попахивает актив рекордом
Юра
Следующий шаг не инджектить вообще EM
Павел
Это все попахивает актив рекордом
Отсутствие uow не равно актив рекорд
Павел
Ну это я так)
Юра
Это все похоже на User::save()
Юра
Разве нет?
Павел
Это все похоже на User::save()
Даже тут нет такого булевого гавна )
Юра
Мы берем центральрую точку входу на БД и оаспихиваем её по всем местам
Павел
Это все похоже на User::save()
Тут юзер в БД лезет, у нас все же отдельный класс
Юра
Ок теперь вместо отдельной транзакции, будет куча флашей
Юра
Каждвй со своей транзакцией
Павел
Ок теперь вместо отдельной транзакции, будет куча флашей
Не совсем поинт понимаю, речь про то, что save в репозитории плохо?
Павел
Запутался что обсуждаем
Юра
И ты слабо это контролируешь потому что кто-то с флагом вызвает кто-то без флага и т.д.
Юра
Короче репозиторий теперь становится чем-то что само решает флашить емк или нет, мне это не нравится ) мое мнение
Павел
Флаш плохо
Это холивар) Например если говорим об агрегатах, там вообще 1 сейв - 1 транзакция дожна быть) Опять таки, если не UoW в системе, то без save в репозитории не обойтись. Нет коллектора изменений. И отсуствтие UoW не равно актив рекорд. Дата маппер то остался. Плюс контракт флашера спорный, что это вообще такое)))
Юра
Точнее решает не само но теперь оно МОЖЕТ флашить, и это затрудняет дебаг и все такое
Павел
Короче репозиторий теперь становится чем-то что само решает флашить емк или нет, мне это не нравится ) мое мнение
Концепция того, что кладем в репо, но это не сохраняется - тоже не однозначная
Юра
Вызывай флаш и оно сохранит все либо ничего
Павел
Вызывай флаш и оно сохранит все либо ничего
Что такое флаш? Ну в не в доктрине а в приложении.