DS
Юзать так
Size: a a a
2020 September 29
DS
Правда CORS заголовков не будет, до мидлтвари навешенной на группу не дойдет
АН
а как origin и т.д. задать ?
DS
Александр Некрасов
а как origin и т.д. задать ?
Никак, это первое что пришло в голову, для CORS не подходит в таком варианте
АН
может в роута добавить withCors и для группы тоже (для удобства), куда реализуем какой-то интерфейс ?
АН
вместо withOptionsAutoResponse
АН
если будет задан cors для роута, то уже отработает
DS
Александр Некрасов
может в роута добавить withCors и для группы тоже (для удобства), куда реализуем какой-то интерфейс ?
Можно для группы withOptionsMethod(), чтобы всем роутам автоматом OPTIONS добавить. Надо подумать над подходом к COSR, там еще и методы кастомно возвращать можно
АН
Можно для группы withOptionsMethod(), чтобы всем роутам автоматом OPTIONS добавить. Надо подумать над подходом к COSR, там еще и методы кастомно возвращать можно
как вариант
AM
@rustamwin https://github.com/yiisoft/auth-jwt/pull/5 я влил. Там всё норм, но есть ощущение что Manager делает две не совсем связанные штуки.
RM
@rustamwin https://github.com/yiisoft/auth-jwt/pull/5 я влил. Там всё норм, но есть ощущение что Manager делает две не совсем связанные штуки.
Разделить на два менеджера?
AM
ну, нарушение SRP налицо, конечно
AM
но насколько это всё целесообразно делать правильно — не уверен
AM
createToken - это фабрика
AM
getClaims - это репозиторий
RM
factory/repository - мне норм
AM
Я просто не на 100% в контексте, просто посмотрел на имена методов.
AM
Это имеет смысл так делить?
RM
Но эти методы связаны друг с другом с key (JWK)