-
-
Notifications
You must be signed in to change notification settings - Fork 248
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
Refresh Token with Union-Type Returning Handler #597
Comments
yeah the built-in refresh token functionality is not compatible with the union type responses. the refresh token feature was built before microsoft came out with the union type. we don't want to introduce a breaking change to that feature at this point in time. will look in to supporting it in the future. so your options right now would be to roll your own refresh token functionality or not use the union type responses. if the community wants to have a go at it, a PR would be highly welcome 😉 |
Ok, thank you for the quick response! |
i'll leave this issue open so that someone can look in to implementing it. |
Why not simply make |
@tcortega i've forgotten the details of how this was implemented, and don't have the capacity to look in to it anytime soon. if you'd like to have a crack at a PR, that would be great. |
Hi,
in my application, I have a login endpoint using Union-Type Returning Handler, so my class is:
I have an issue when I try to use JWT Refresh Token Service to implement my refresh token endpoint and logic: the refresh endpoint response type can't be converted to login endpoint response type, because it's a union of typed result.
Is there a way to do something similar?
For the moment, I have to code both login and refresh endpoint independently, without using the token service.
The text was updated successfully, but these errors were encountered: