コードの読みやすさを向上させるために、ログインコントローラのリファクタリングを検討してきました。そうすることで、私は今、これは私がダウンロード可能な例から使用したものよりもヒープは、クリーンで読みやすくなります。このDotNetOpenAuth - "View"はこれとどのようにやりとりしますか
using DotNetOpenAuth.Messaging;
public ActionResult LogOn()
{
var openid = new OpenIdRelyingParty();
IAuthenticationResponse response = openid.GetResponse();
if (response != null)
{
switch (response.Status)
{
case AuthenticationStatus.Authenticated:
FormsAuthentication.RedirectFromLoginPage(
response.ClaimedIdentifier, false);
break;
case AuthenticationStatus.Canceled:
ModelState.AddModelError("loginIdentifier",
"Login was cancelled at the provider");
break;
case AuthenticationStatus.Failed:
ModelState.AddModelError("loginIdentifier",
"Login failed using the provided OpenID identifier");
break;
}
}
return View();
}
[System.Web.Mvc.AcceptVerbs(HttpVerbs.Post)]
public ActionResult LogOn(string loginIdentifier)
{
if (!Identifier.IsValid(loginIdentifier))
{
ModelState.AddModelError("loginIdentifier",
"The specified login identifier is invalid");
return View();
}
else
{
var openid = new OpenIdRelyingParty();
IAuthenticationRequest request = openid.CreateRequest(
Identifier.Parse(loginIdentifier));
// Require some additional data
request.AddExtension(new ClaimsRequest
{
BirthDate = DemandLevel.NoRequest,
Email = DemandLevel.Require,
FullName = DemandLevel.Require
});
return request.RedirectingResponse.AsActionResult();
}
}
のように見えますProgrammatic OpenID Relying Party exampleに出くわしました。 (私は最新バージョンをダウンロードし、これは彼らが与える例である - 。私は5ヶ月前に私のアプリを構築して同じ例である)
[ValidateInput(false)]
public ActionResult Authenticate(string returnUrl) {
var response = openid.GetResponse();
if (response == null) {
// Stage 2: user submitting Identifier
Identifier id;
if (Identifier.TryParse(Request.Form["openid_identifier"], out id)) {
try {
return openid.CreateRequest(Request.Form["openid_identifier"]).RedirectingResponse.AsActionResult();
} catch (ProtocolException ex) {
ViewData["Message"] = ex.Message;
return View("Login");
}
} else {
ViewData["Message"] = "Invalid identifier";
return View("Login");
}
} else {
// Stage 3: OpenID Provider sending assertion response
switch (response.Status) {
case AuthenticationStatus.Authenticated:
Session["FriendlyIdentifier"] = response.FriendlyIdentifierForDisplay;
FormsAuthentication.SetAuthCookie(response.ClaimedIdentifier, false);
if (!string.IsNullOrEmpty(returnUrl)) {
return Redirect(returnUrl);
} else {
return RedirectToAction("Index", "Home");
}
case AuthenticationStatus.Canceled:
ViewData["Message"] = "Canceled at provider";
return View("Login");
case AuthenticationStatus.Failed:
ViewData["Message"] = response.Exception.Message;
return View("Login");
}
}
return new EmptyResult();
}
今、その例は、私の好みのためであれば、あまりにも多くの文を持っており、私が追加しなければならない余分な処理(activity logging
とchecking for new user
またはadd to existing account
)と一緒に、それは本当に面倒で本当に速くなり始めます。
残念ながら、私のコードを最初の例のようにリファクタリングすると、小さな問題が残っています。ビューはこれとどのように相互作用しますか?つまり、openid.GetResponse()
を探していますが、その回答をどのように提出しますか?
私が言ったように、私はこの作業を得ることができます、それは私の現在の方法よりもはるかにクリーンであるように見えます。
最後の行はどのように機能しましたか? 'return request.RedirectingResponse.AsActionResult();' 私はいくつかの名前空間の使用法を追加しましたが、私はこの部分をどのように進めるのかを考えることができません。 HttpRequestBaseにはRedirectingResponseの参照が含まれていません。 – zeristor