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.

Sign up for GitHub

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

Jump to bottom

Version 6.3.0 - Redirection After Login #743

Open
abald124 opened this issue Sep 22, 2017 · 3 comments · May be fixed by #763
Open

Version 6.3.0 - Redirection After Login #743

abald124 opened this issue Sep 22, 2017 · 3 comments · May be fixed by #763

Comments

@abald124
Copy link

Upgraded to version 6.3.0 and using Active Directory Membership Service . Having issues with the redirect after authentication similar to the issues described in #731 and #733.

On a successful login the URL in address bar changes to /Home/Logon?returnUrl=%2FBonobo.Git.Server%2FHome%2FIndex but still stuck on the login page.

Know login was successful because I can manually put /Account in the address bar and get to the accounts index view.
@CatStarwind
Copy link
Contributor

I'm not sure why, but going to "/Home/WindowsLogin" solves this.

@CatStarwind
Copy link
Contributor

Still looking into this, so far it seems the issue stems from this commit. Thus far I was able to patch it by changing line 44 in RepositoryController.cs to the following:

return RedirectToAction((AuthenticationProvider is WindowsAuthenticationProvider ? "WindowsLogin" : "Logon"), "Home", new { returnUrl = Url.Action("index", "Home") });

But I'm still looking into a more elegant solution/why it doesn't authenticate before hand. It doesn't seem like the CookieAuthenticationOptions that are configured ever get used.

@CatStarwind CatStarwind linked a pull request Nov 21, 2017 that will close this issue
Open
@CatStarwind CatStarwind mentioned this issue Feb 1, 2018
Open
@AlmightyCZ
Copy link

Was there any release fixing this issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added hotfix for initial WindowsAuthentication use. CatStarwind/Bonobo-Git-Server
3 participants
@abald124 @CatStarwind @AlmightyCZ