6

我正在尝试通过远程身份验证服务对用户进行身份验证。我已经编写了将消息发送到服务并等待结果的辅助方法:

def authenticateAwait(email:       String,
                      password:    String
                     ): Either[String, Option[User]] = {
  try {
    val future = authenticate(email, password)
    Right(Await.result(future, timeout.duration))
  } catch {
    case _ ⇒ Left("Unable to connect to authentication server")
  }
}

Left[String]如果无法发送消息或没有响应,则返回错误描述。如果收到服务响应,则返回Right[Option[User]]Option[User]服务根据身份验证结果进行响应。

为了执行实际的身份验证,我使用几个验证器创建了表单,这里是:

val loginForm = Form(
  tuple(
    "email"    → email,
    "password" → nonEmptyText
  ) verifying ("Invalid email or password", result => result match {
    case (email, password) ⇒
      User.authenticateAwait(email, password) match {
        case Left(_) ⇒ true
        case Right(optUser) ⇒ optUser.isDefined
      }
  }) verifying ("Unable to connect to authentication server", result => result match {
    case (email, password) ⇒
      User.authenticateAwait(email, password) match {
        case Left(_) ⇒ false
        case Right(optUser) ⇒ true
      }
  })
)

有一件事让我担心这段代码,它调用authenticateAwait了两次。这意味着每次验证将发送两条消息。我真正需要的是调用authenticateAwait一次,存储结果并对其执行各种验证。似乎没有简单的解决方案。

要执行身份验证,访问所需的表单字段,这意味着应该绑定然后验证表单,但是没有办法将错误附加到现有表单(我错了吗?)。

错误只能在创建过程中附加到表单,因此我应该在验证器中执行身份验证,但随后会出现上述问题。

我带来的临时解决方案是在其中定义一个方法和一个var

def loginForm = {
  var authResponse: Either[String, Option[commons.User]] = null

  Form(
    tuple(
      "email"    → email,
      "password" → nonEmptyText
    ) verifying ("Invalid email or password", result ⇒ result match {
      case (email, password) ⇒
        authResponse = User.authenticateAwait(email, password)
        authResponse match {
          case Left(_) ⇒ true
          case Right(optUser) ⇒ optUser.isDefined
        }
    }) verifying ("Unable to connect to authentication server", result ⇒ result match {
      case (email, password) ⇒
        authResponse match {
          case Left(_) ⇒ false
          case Right(optUser) ⇒ true
        }
    })
  )
}

这显然是一个 hack。有没有更好的解决方案?

更新: 在我看来,表单应该只清理输入,但稍后应该在表单之外执行身份验证。问题是错误作为 的一部分发送到视图,Form并且不可能将错误附加到现有表单。也没有简单的方法可以创建带有错误的新表单。

4

2 回答 2

3

您必须了解的是 Form 是不可变的。但是有一个易于使用的实用方法来构造一个添加了错误的新表单:

loginForm.copy(errors = Seq(FormError("email", "Already registered")))
于 2012-10-03T06:01:12.947 回答
0

当然,将身份验证与验证混为一谈只会使简单的操作变得复杂。下面是未经测试的,但这是我要进入的方向,正确的预测通过理解过滤。

// point of validation is to sanitize inputs last I checked
val form = Form(tuple("email"→ email, "password"→ nonEmptyText)
val res = for{
  case(e,p) <- form.bindFromRequest.toRight("Invalid email or password")
  success   <- User.authenticateAwait(e,p).right 
} yield success
res fold( Conflict(Left(_)), u=> Ok(Right(u)) )
于 2012-09-27T11:20:58.020 回答