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

Web Login only works on fresh instance of browser with no cache #167

Closed
Kronos11 opened this issue Sep 4, 2020 · 6 comments
Closed

Web Login only works on fresh instance of browser with no cache #167

Kronos11 opened this issue Sep 4, 2020 · 6 comments
Assignees
Labels
Authentication bug Something isn't working
Milestone

Comments

@Kronos11
Copy link
Contributor

Kronos11 commented Sep 4, 2020

Authentication doesn't seem to work properly using AAD except if you clear the cache/session data.
Is something being stored that needs to be removed upon a new login attempt?

@unaizorrilla
Copy link
Collaborator

On 4.0 preview? Let me check this weekend!

@Kronos11
Copy link
Contributor Author

Kronos11 commented Sep 4, 2020

Yeah sorry on 4.0 preview

@unaizorrilla
Copy link
Collaborator

Are you debugging with VS? Can you try with a compose and test it without debug?

@unaizorrilla
Copy link
Collaborator

I'm debugging the issue, the EvaluatingAuthorization on Authentication.razor seems that throw on get my info and a infinite loop is produced! Catching the exception and move to not allowed looks good but I'm testing it

@unaizorrilla unaizorrilla added this to the Esquio-4 milestone Sep 4, 2020
@unaizorrilla unaizorrilla added Authentication bug Something isn't working labels Sep 4, 2020
@unaizorrilla unaizorrilla self-assigned this Sep 4, 2020
unaizorrilla added a commit that referenced this issue Sep 5, 2020
@unaizorrilla
Copy link
Collaborator

Hi @Kronos11

Just added a commit that work around this issue ¿can you test it?

I prepare a new compose file for aad to test without debug, please close the issue if all is ok!

https://github.com/Xabaril/Esquio/blob/master/build/docker-compose-demo-with-ui-sqlserver.aad.yml

If works well we can release a new preview this week and well, test all scenarios to release the 4.0 final version!

Thanks for your help testing Esquio!!
Unai

@Kronos11
Copy link
Contributor Author

Kronos11 commented Sep 5, 2020

Validated it is working as expected! Thanks for the quick fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Authentication bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants