Custom RemoteAuthenticationHandler Exceptions

[nquandt]

Is there an existing issue for this?

• I have searched the existing issues

Is your feature request related to a problem? Please describe the problem.

I would like to be able to handle exceptions thrown by AuthenticationHandlers, specifically RemoteAuthenticationHandlers. Generic example. Say something when wrong with authentication and I'd like to clear cookies.

aspnetcore/src/Security/Authentication/Core/src/HandleRequestResult.cs

Line 68 in b6bd323

=> Fail(new Exception(failureMessage));

aspnetcore/src/Security/Authentication/Core/src/RemoteAuthenticationHandler.cs

Line 130 in b6bd323

throw new Exception("An error was encountered while handling the remote login.", errorContext.Failure);

Describe the solution you'd like

Either a custom exception type so that we can create middleware to catch

try {
    await next()
catch (AuthenticationHandlerException ex) {
    Do stuff
}

or some way of plugging into the handling of errors.

Additional context

No response

[Tratcher]

See here:

aspnetcore/src/Security/Authentication/Core/src/RemoteAuthenticationHandler.cs

Lines 106 to 110 in b6bd323

	var errorContext = new RemoteFailureContext(Context, Scheme, Options, exception)
	{
	Properties = properties
	};
	await Events.RemoteFailure(errorContext);

aspnetcore/src/Security/Authentication/Core/src/Events/RemoteAuthenticationEvents.cs

Line 34 in b6bd323

public virtual Task RemoteFailure(RemoteFailureContext context) => OnRemoteFailure(context);

aspnetcore/src/Security/Authentication/Core/src/RemoteAuthenticationOptions.cs

Line 122 in b6bd323

public new RemoteAuthenticationEvents Events

[ghost]

Hi @nquandt. We have added the "Needs: Author Feedback" label to this issue, which indicates that we have an open question for you before we can take further action. This issue will be closed automatically in 7 days if we do not hear back from you by then - please feel free to re-open it if you come back to this issue after that time.

[nquandt]

Thanks @Tratcher! This was exactly what I was looking for, I totally missed that.