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

Is refresh token required? #92

Open
rob-burrus opened this issue Apr 29, 2019 · 1 comment
Open

Is refresh token required? #92

rob-burrus opened this issue Apr 29, 2019 · 1 comment

Comments

@rob-burrus
Copy link

I'm trying to authenticate with ADFS. Unfortunately, if we extend the access token's lifetime to be longer than the refresh token's lifetime, ADFS won't send a refresh token at all (makes sense). We're only able to change access token lifetime per application, and not the refresh token lifetime (horrible limitation by our version of ADFS). So I'm wondering if I can just ignore the refresh token completely? This nuget seems to require that there be a refresh token, else it keeps prompting the user to sign in. Any advice?

@Clancey
Copy link
Owner

Clancey commented Apr 29, 2019

Out of the box no, this is not typical. I really try to support the normal stuff. However it is super easy to extend and make it work how you want. You just need to override: https://github.com/Clancey/SimpleAuth/blob/master/src/SimpleAuth/OAuth/OAuthAccount.cs#L39 as well as the method that loads it, https://github.com/Clancey/SimpleAuth/blob/master/src/SimpleAuth/Api/AuthenticatedApi.cs#L88

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

No branches or pull requests

2 participants