diff options
author | Andrew Arnott <andrewarnott@gmail.com> | 2010-10-06 21:31:06 -0700 |
---|---|---|
committer | Andrew Arnott <andrewarnott@gmail.com> | 2010-10-06 21:31:06 -0700 |
commit | 6092653f54bcca908643bdf535b3cb0c2b7281e7 (patch) | |
tree | 2eaae6841806a08d7aabfff412a339e93a7a78b5 /samples/OpenIdProviderMvc/Controllers/OpenIdController.cs | |
parent | d4477a5d0020ada7b4d13469ad4908860edcb39a (diff) | |
download | DotNetOpenAuth-6092653f54bcca908643bdf535b3cb0c2b7281e7.zip DotNetOpenAuth-6092653f54bcca908643bdf535b3cb0c2b7281e7.tar.gz DotNetOpenAuth-6092653f54bcca908643bdf535b3cb0c2b7281e7.tar.bz2 |
Improved OpenID Provider MVC sample to support immediate mode and anonymous OpenID requests.
Diffstat (limited to 'samples/OpenIdProviderMvc/Controllers/OpenIdController.cs')
-rw-r--r-- | samples/OpenIdProviderMvc/Controllers/OpenIdController.cs | 191 |
1 files changed, 160 insertions, 31 deletions
diff --git a/samples/OpenIdProviderMvc/Controllers/OpenIdController.cs b/samples/OpenIdProviderMvc/Controllers/OpenIdController.cs index bd0fdbf..0ef9680 100644 --- a/samples/OpenIdProviderMvc/Controllers/OpenIdController.cs +++ b/samples/OpenIdProviderMvc/Controllers/OpenIdController.cs @@ -9,63 +9,192 @@ namespace OpenIdProviderMvc.Controllers { using DotNetOpenAuth.OpenId; using DotNetOpenAuth.OpenId.Behaviors; using DotNetOpenAuth.OpenId.Extensions.ProviderAuthenticationPolicy; + using DotNetOpenAuth.OpenId.Extensions.SimpleRegistration; using DotNetOpenAuth.OpenId.Provider; using OpenIdProviderMvc.Code; public class OpenIdController : Controller { internal static OpenIdProvider OpenIdProvider = new OpenIdProvider(); - internal static IAuthenticationRequest PendingAuthenticationRequest { - get { return ProviderEndpoint.PendingAuthenticationRequest; } - set { ProviderEndpoint.PendingAuthenticationRequest = value; } - } - [ValidateInput(false)] public ActionResult Provider() { IRequest request = OpenIdProvider.GetRequest(); if (request != null) { - var authRequest = request as IAuthenticationRequest; - if (authRequest != null) { - PendingAuthenticationRequest = authRequest; - if (authRequest.IsReturnUrlDiscoverable(OpenIdProvider) == RelyingPartyDiscoveryResult.Success && - User.Identity.IsAuthenticated && - (authRequest.IsDirectedIdentity || this.UserControlsIdentifier(authRequest))) { - return this.SendAssertion(); - } else { - return RedirectToAction("LogOn", "Account", new { returnUrl = Url.Action("SendAssertion") }); - } - } - + // Some requests are automatically handled by DotNetOpenAuth. If this is one, go ahead and let it go. if (request.IsResponseReady) { return OpenIdProvider.PrepareResponse(request).AsActionResult(); - } else { - return RedirectToAction("LogOn", "Account"); } + + // This is apparently one that the host (the web site itself) has to respond to. + ProviderEndpoint.PendingRequest = (IHostProcessedRequest)request; + + // Try responding immediately if possible. + ActionResult response; + if (this.AutoRespondIfPossible(out response)) { + return response; + } + + // We can't respond immediately with a positive result. But if we still have to respond immediately... + if (ProviderEndpoint.PendingRequest.Immediate) { + // We can't stop to prompt the user -- we must just return a negative response. + return this.SendAssertion(); + } + + return this.RedirectToAction("AskUser"); } else { - return View(); + // No OpenID request was recognized. This may be a user that stumbled on the OP Endpoint. + return this.View(); } } + /// <summary> + /// Displays a confirmation page. + /// </summary> [Authorize] + public ActionResult AskUser() { + if (ProviderEndpoint.PendingRequest == null) { + // Oops... precious little we can confirm without a pending OpenID request. + return this.RedirectToAction("Index", "Home"); + } + + // The user MAY have just logged in. Try again to respond automatically to the RP if appropriate. + ActionResult response; + if (this.AutoRespondIfPossible(out response)) { + return response; + } + + this.ViewData["Realm"] = ProviderEndpoint.PendingRequest.Realm; + + return this.View(); + } + + [HttpPost, Authorize, ValidateAntiForgeryToken] + public ActionResult AskUserResponse(bool confirmed) { + if (ProviderEndpoint.PendingAnonymousRequest != null) { + ProviderEndpoint.PendingAnonymousRequest.IsApproved = confirmed; + } else if (ProviderEndpoint.PendingAuthenticationRequest != null) { + ProviderEndpoint.PendingAuthenticationRequest.IsAuthenticated = confirmed; + } else { + throw new InvalidOperationException("There's no pending authentication request!"); + } + + return this.SendAssertion(); + } + + /// <summary> + /// Sends a positive or a negative assertion, based on how the pending request is currently marked. + /// </summary> + /// <returns>An MVC redirect result.</returns> public ActionResult SendAssertion() { - IAuthenticationRequest authReq = PendingAuthenticationRequest; - PendingAuthenticationRequest = null; // clear session static so we don't do this again - if (authReq == null) { + var pendingRequest = ProviderEndpoint.PendingRequest; + var authReq = pendingRequest as IAuthenticationRequest; + var anonReq = pendingRequest as IAnonymousRequest; + ProviderEndpoint.PendingRequest = null; // clear session static so we don't do this again + if (pendingRequest == null) { throw new InvalidOperationException("There's no pending authentication request!"); } - if (authReq.IsDirectedIdentity) { - authReq.LocalIdentifier = Models.User.GetClaimedIdentifierForUser(User.Identity.Name); + // Set safe defaults if somehow the user ended up (perhaps through XSRF) here before electing to send data to the RP. + if (anonReq != null && !anonReq.IsApproved.HasValue) { + anonReq.IsApproved = false; } - if (!authReq.IsDelegatedIdentifier) { - authReq.ClaimedIdentifier = authReq.LocalIdentifier; + + if (authReq != null && !authReq.IsAuthenticated.HasValue) { + authReq.IsAuthenticated = false; } - // Respond to AX/sreg extension requests. - //// Real web sites would have code here + if (authReq != null && authReq.IsAuthenticated.Value) { + if (authReq.IsDirectedIdentity) { + authReq.LocalIdentifier = Models.User.GetClaimedIdentifierForUser(User.Identity.Name); + } + + if (!authReq.IsDelegatedIdentifier) { + authReq.ClaimedIdentifier = authReq.LocalIdentifier; + } + } + + // Respond to AX/sreg extension requests only on a positive result. + if ((authReq != null && authReq.IsAuthenticated.Value) || + (anonReq != null && anonReq.IsApproved.Value)) { + // Look for a Simple Registration request. When the AXFetchAsSregTransform behavior is turned on + // in the web.config file as it is in this sample, AX requests will come in as SReg requests. + var claimsRequest = pendingRequest.GetExtension<ClaimsRequest>(); + if (claimsRequest != null) { + var claimsResponse = claimsRequest.CreateResponse(); + + // This simple respond to a request check may be enhanced to only respond to an individual attribute + // request if the user consents to it explicitly, in which case this response extension creation can take + // place in the confirmation page action rather than here. + if (claimsRequest.Email != DemandLevel.NoRequest) { + claimsResponse.Email = User.Identity.Name + "@dotnetopenauth.net"; + } + + pendingRequest.AddResponseExtension(claimsResponse); + } + } + + return OpenIdProvider.PrepareResponse(pendingRequest).AsActionResult(); + } + + /// <summary> + /// Attempts to formulate an automatic response to the RP if the user's profile allows it. + /// </summary> + /// <param name="response">Receives the ActionResult for the caller to return, or <c>null</c> if no automatic response can be made.</param> + /// <returns>A value indicating whether an automatic response is possible.</returns> + private bool AutoRespondIfPossible(out ActionResult response) { + // If the odds are good we can respond to this one immediately (without prompting the user)... + if (ProviderEndpoint.PendingRequest.IsReturnUrlDiscoverable(OpenIdProvider) == RelyingPartyDiscoveryResult.Success + && User.Identity.IsAuthenticated + && this.HasUserAuthorizedAutoLogin(ProviderEndpoint.PendingRequest)) { + // Is this is an identity authentication request? (as opposed to an anonymous request)... + if (ProviderEndpoint.PendingAuthenticationRequest != null) { + // If this is directed identity, or if the claimed identifier being checked is controlled by the current user... + if (ProviderEndpoint.PendingAuthenticationRequest.IsDirectedIdentity + || this.UserControlsIdentifier(ProviderEndpoint.PendingAuthenticationRequest)) { + ProviderEndpoint.PendingAuthenticationRequest.IsAuthenticated = true; + response = this.SendAssertion(); + return true; + } + } + + // If this is an anonymous request, we can respond to that too. + if (ProviderEndpoint.PendingAnonymousRequest != null) { + ProviderEndpoint.PendingAnonymousRequest.IsApproved = true; + response = this.SendAssertion(); + return true; + } + } + + response = null; + return false; + } + + /// <summary> + /// Determines whether the currently logged in user has authorized auto login to the requesting relying party. + /// </summary> + /// <param name="request">The incoming request.</param> + /// <returns> + /// <c>true</c> if it is safe to respond affirmatively to this request and all extensions + /// without further user confirmation; otherwise, <c>false</c>. + /// </returns> + private bool HasUserAuthorizedAutoLogin(IHostProcessedRequest request) { + // TODO: host should implement this method meaningfully, consulting their user database. + // Make sure the user likes the RP + if (true/*User.UserLikesRP(request.Realm))*/) { + // And make sure the RP is only asking for information about the user that the user has granted before. + if (true/*User.HasGrantedExtensions(request)*/) { + // For now for the purposes of the sample, we'll disallow auto-logins when an sreg request is present. + if (request.GetExtension<ClaimsRequest>() != null) { + return false; + } + + return true; + } + } - authReq.IsAuthenticated = this.UserControlsIdentifier(authReq); - return OpenIdProvider.PrepareResponse(authReq).AsActionResult(); + // If we aren't sure the user likes this site and is willing to disclose the requested info, return false + // so the user has the opportunity to explicity choose whether to share his/her info. + return false; } /// <summary> |