Closed armpogart closed 3 years ago
Should be moved to https://github.com/yiisoft/user
The issue description is updated with corresponding PR links.
Concerning the last point I'm not sure whether we should remove that method from UserRepository
or just implement the new interface.
Concerning the last point I'm not sure whether we should remove that method from
UserRepository
or just implement the new interface.
Just remove. Someone has already removed the token
field from the User
class and the findIdentityByToken
method is not used.
Merged all the PRs linked. What's left, @armpogart?
@samdark yiisoft/auth#38 and yiisoft/yii-demo#224 If I haven't forgotten anything, after merging those 2 PRs the issue can be closed.
All merged. Thanks for hadling it.
IdentityRepositoryInterface::findIdentityByToken
(yiisoft/auth#37)IdentityRepositoryInterface
with the token (yiisoft/auth#37)Yiisoft\Auth\Method\HttpBasic
,Yiisoft\Auth\Method\HttpHeader
,Yiisoft\Auth\Method\QueryParam
to depend on the new interface. (yiisoft/auth#37)User::loginByAccessToken
functionality altogether. We don't need it there for anything. I don't see any code that specifically uses it. And I don't see any reason we have to provide such API whenYiisoft\Auth\Middleware\Auth
can do that with its' respectiveAuthInterface
implementations (mentioned above). (#20)App\Repository\UserRepository
) (yiisoft/yii-demo#224)