Conversation
|
Thank you very much for this pull request @jackywxd! Sorry it took so long to get back to it. I think this is a great start. My questions would be
|
|
@daffl David, thanks for your time! You are right, it can be on its own module and it is a much cleaner and easier solution. The current code change is only applied for server-side, not related to Anyway, I will close this request for now, will see if I can make changes to both server and client and publish as separated modules. Thanks again! |
|
Great, thank you. Please let me know if you need anything. Once it is ready we should definitely also do a writeup. |
|
I'm looking for it 👍 |
|
@jackywxd! I found only the TypeScript version of the "refresh token " fork , do we have the Java script version also? Thanks for your help on this. |
Summary
Enable refresh-token server side support by extending the existing Authentication Service. As refresh-token is a stateful resource, it also depends on refresh-token entity service, similar as local strategy depends on user entity service.
Related issue: #1337
Includes unit test cases