active-directory-b2c-dotnet-webapp-and-webapi icon indicating copy to clipboard operation
active-directory-b2c-dotnet-webapp-and-webapi copied to clipboard

Configure code to run at non-root context

Open phillipjohnson opened this issue 3 years ago • 2 comments

In the Account controller, the redirect URL is hard-coded to root, but I believe it should actually pull from the config to accommodate applications running at a given context other than root. Here is my modification:

public void SignUpSignIn(string redirectUrl)
{
    redirectUrl = redirectUrl ?? Utils.Globals.RedirectUri; // Important!

    // Use the default policy to process the sign up / sign in flow
    HttpContext.GetOwinContext().Authentication.Challenge(new AuthenticationProperties { RedirectUri = redirectUrl });
    return;
}

There are also many Response.Redirect lines that I think should be changed to start with ~/ instead of just /. Let me know if PRs are welcome and I can submit one.

phillipjohnson avatar Jun 30 '21 20:06 phillipjohnson

@phillipjohnson : thanks for raising PRs are definitively welcome. Thanks for proposing

jmprieur avatar Jul 05 '21 09:07 jmprieur

Thanks, for some reason it looks like sandbox Azure AD is not working so I'll have to wait a bit on this.

phillipjohnson avatar Jul 06 '21 15:07 phillipjohnson