30

I'm using OWIN / OAuth with OpenId Connect authentication (Microsoft.Owin.Security.OpenIdConnect) in a C# ASP MVC web app. The SSO login with Microsoft account basically works, but from time to time I'm getting an error page on the browser that says Bad Request - Request Too Long.

I found out that this error is caused by too many cookies. Deleting cookies helps for some time, but after a while the problem comes back.

The cookies that cause the problem are set from OpenId framework, so there are dozens of cookies with names like OpenIdConnect.nonce.9oEtF53WxOi2uAw........

This is not SPA application, but some parts are refreshed periodically with ajax calls.

andrew.fox
  • 7,435
  • 5
  • 52
  • 75

4 Answers4

57

It turned out that the root cause was the Ajax call.

The problematic flow was

1) OAuth cookie got expired after some time

2) Expiration normally causes redirection the page to login.microsoft.com to refresh the cookie. In this step OAuth framework adds new nonce cookie to the response (every time)!

3) But Ajax doesn't handle redirections outside of the domain (cross-domain to login.microsoft.com). But the cookie was already appended to the page.

4) Next periodical Ajax call repeated the flow causing rapid increase of 'nonce' cookies.

Solution

I had to extend the "OWIN OpenId" framework setup code to handle Ajax calls differently - to prevent redirection and stop sending cookies.

public void ConfigureAuth(IAppBuilder app)
{
    app.SetDefaultSignInAsAuthenticationType(CookieAuthenticationDefaults.AuthenticationType);
    app.UseCookieAuthentication(new CookieAuthenticationOptions());

    app.UseOpenIdConnectAuthentication(
        new OpenIdConnectAuthenticationOptions
        {
            ClientId = clientId,
            Authority = authority,
            Notifications = new OpenIdConnectAuthenticationNotifications
            {
                RedirectToIdentityProvider = ctx => 
                {
                    bool isAjaxRequest = (ctx.Request.Headers != null && ctx.Request.Headers["X-Requested-With"] == "XMLHttpRequest");

                    if (isAjaxRequest)
                    {
                        ctx.Response.Headers.Remove("Set-Cookie");
                        ctx.State = NotificationResultState.HandledResponse;
                    }

                    return Task.FromResult(0);
                }
            }
        });
}

The Ajax caller had to be adjusted too to detect 401 code and perform full page refresh (which caused a quick redirect to Microsoft authority).

andrew.fox
  • 7,435
  • 5
  • 52
  • 75
8

For me the solution was to enforce the creation of an ASP.NET session.

Steps to reproduce:

  • Delete ASP.NET_SessionId cookie and idsrv cookie on a protected page of your webapp
  • Reload page
  • Redirect to OIDC authentication store and authenticate
  • Redirect back to webapp => validation of the authentication fails, because no asp.net session is available
  • Endless redirects until 'Request too long...' error happens

Solution: Enforce session creation by adding

protected void Session_Start(object sender, EventArgs e)
{
}

to global.asax.cs.

merrycoder
  • 111
  • 1
  • 5
1

OWIN and MVC may be deleting each other's cookies as described by the AspNetKatana github.

  • As a workaround that page suggests to explicitly use SystemWebCookieManager or SystemWebChunkingCookieManager (Microsoft.Owin.Host.SystemWeb 3.1.0).
  • There is also the Kentor Owin Cookie Saver as a workaround.
  • However, I would first try upgrading to Owin 4.1.0 (released november 2019), as this seems to fix it!
Yahoo Serious
  • 3,728
  • 1
  • 33
  • 37
0

In my case the problem was the order in which I was configuring the application inside Startup.cs.

Reminder to self - always configure authentication first!

public class Startup
{
    public void Configuration(IAppBuilder app)
    {
        app.SetDefaultSignInAsAuthenticationType(CookieAuthenticationDefaults.AuthenticationType);
        app.UseCookieAuthentication(new CookieAuthenticationOptions());
        app.UseOpenIdConnectAuthentication(
            new OpenIdConnectAuthenticationOptions
            {
                ClientId = _clientId,
                ClientSecret = _clientSecret,
                Authority = _authority,
                RedirectUri = _redirectUri
            });

        // configure the rest of the application...
    }
peco
  • 3,890
  • 1
  • 20
  • 27