Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable refresh-token server side support #2026

Closed
wants to merge 6 commits into from

Conversation

jackywxd
Copy link

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

@daffl
Copy link
Member

daffl commented Aug 18, 2020

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

  • If it can be its own module, it is always easier to publish it separately instead of pulling it into the core
  • Aren't there any changes that need to be made in the core modules (like the authentication-client)?

@jackywxd
Copy link
Author

@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 authentication-client. I have't got a chance to look at the client-side code yet. Ideally, the client-side integration should be included.

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!

@jackywxd jackywxd closed this Aug 24, 2020
@daffl
Copy link
Member

daffl commented Aug 24, 2020

Great, thank you. Please let me know if you need anything. Once it is ready we should definitely also do a writeup.

@itoonx
Copy link

itoonx commented Aug 28, 2020

I'm looking for it 👍

@bujji1
Copy link

bujji1 commented Dec 6, 2021

@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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants