DS
реюзабельно можно сделать, кстати, как @roxblnfk говорит
C диспетчера middleware сделать?
Size: a a a
2020 February 19
AM
C диспетчера middleware сделать?
да, думаю, стоит вернуть
AM
@roxblnfk интересная идея на тему переименовать в pipeline
AM
имя неплохое
AM
вероятно, понятней, чем диспетчер
AM
Chain всё ☠️
DS
да, думаю, стоит вернуть
Вернул, остальное тоже поправил
DS
Что с наименованием метода добавления middleware, я поменял на addMiddleware(), если будем переименовывать в PipeLine, то так точно лучше.
DS
Я это у всех на виде на рейтлимитер тесты не проходят?
AM
Я это у всех на виде на рейтлимитер тесты не проходят?
У меня норм, всё зелёное.
AM
Что с наименованием метода добавления middleware, я поменял на addMiddleware(), если будем переименовывать в PipeLine, то так точно лучше.
addMiddleware норм
AM
хотя...
AM
там же callable...
DS
Осталось с переименованием класса/методов порешать. Если оставлять наименование диспетчера как сейчас, то и метод для запуска пайплайна должен называться dispatch(), потому как диспетчер без такого метода будет выглядеть странно)
AM
и, кстати, нужен ли он там...
DS
addMiddleware норм
Ну у нас так и в Route называется, думаю лучше одинаково называть)
DS
и, кстати, нужен ли он там...
dispatch() ? Ну, кагбэ да, фолбек хендлер не надо задавать обязательно
А
dispatch() ? Ну, кагбэ да, фолбек хендлер не надо задавать обязательно
никто не будет использовать этот метод. Он один раз в приложении прописан и всё
А
почему бы не использовать тогда handle()?