Телеграмм чат группы yii3ru страница 1700

Но интерфейс токена не должен быть в yii-view пакете

почему?

12:27пожаловаться #1

СП

Не логично.

"В CsrfMiddleware в конструкторе получаешь CsrfTokenInterface и делаешь $csrf->setToken(...). Теперь токен доступен в контейнере"

Получается yii-web будет зависеть от yii-view чтобы получить интефейс токена.

12:27пожаловаться #2

Ну значит не делаешь в yii-web

12:28пожаловаться #3

СП

тогда зачем метод set)

12:28пожаловаться #4

СП

В рамках yii-view нужен только get и всё

12:28пожаловаться #5

тогда зачем метод set)

Для других реализаций

12:28пожаловаться #6

СП

Dmitriy S

Для других реализаций

так это специфика реализации - кто хочет пусть пилит, зачем в интерфейсе

Group route parameters in a Request attributes · Issue #55 · yiisoft/router

12:29пожаловаться #7

Alexander Makarov in Yii Framework 3

https://github.com/yiisoft/router/issues/55 @roxblnfk актуально?

GitHub

I believe that it would be more correct if the router writes route parameters to its group in the attributes, and not as it is now Route::get('/test/{id:\w+}/{param1}[/{param2}]', ....

12:30пожаловаться #8

так это специфика реализации - кто хочет пусть пилит, зачем в интерфейсе

Сделал я свой вариант CsrfMiddleware и хочу сетить токен не через атрибут, а в контейнер. Как мне быть?

12:30пожаловаться #9

СП

Dmitriy S

Сделал я свой вариант CsrfMiddleware и хочу сетить токен не через атрибут, а в контейнер. Как мне быть?

Зачем тебе для этого метод set в интерфейсе?

12:31пожаловаться #10

СП

Сделал в своей реализации set и засеттил

12:31пожаловаться #11

Сделал в своей реализации set и засеттил

Это нарушит интерфейс

12:32пожаловаться #12

СП

Dmitriy S

Это нарушит интерфейс

А если ты захочешь сделать в токене ещё генерацию? добавить в интерфейс метод generate?

12:32пожаловаться #13

СП

Это всё не относится к yii-view

12:33пожаловаться #14

СП

Во ViewRenderer:
public function withCsrf(string $requestAttribute = null): self

А надо ли тут вообще передавать requestAttribute ?

12:38пожаловаться #15

СП

Выпилить это тогда и сделать с интерфейсом get и set

Group route parameters in a Request attributes · Issue #55 · yiisoft/router

12:39пожаловаться #16

Алексей R in Yii Framework 3

Alexander Makarov

https://github.com/yiisoft/router/issues/55 @roxblnfk актуально?

GitHub

I believe that it would be more correct if the router writes route parameters to its group in the attributes, and not as it is now Route::get('/test/{id:\w+}/{param1}[/{param2}]', ....

не знаю. исправлено?

12:50пожаловаться #17

СП

Add implementation of CsrfViewInjectionInterface. by vjik · Pull Request #2 · yiisoft/yii-view

@yiiliveext запилил с интерфейсом CsrfTokenInterface: https://github.com/yiisoft/yii-view/pull/2/

GitHub

Q
A

Is bugfix?
❌

New feature?
✔️

Breaks BC?
❌

Fixed issues
-

12:53пожаловаться #18

СП

Но опять проблема... если в проекте используется другой роутер, то yiisoft/router всё-равно подтянется, так как пакет yiisoft/yii-view зависит от него. Это ничего страшного?

13:02пожаловаться #19

Если вынести интерфейс в отдельный пакет, а в CsrfMiddleware использовать setToken вместо withAttribute, то все требования по независимости пакетов друг от друга будут соблюдены