cakephp - Cakephp 2.4.x。 Auth 组件 login() 始终返回 true

Question

很抱歉问了一个很长的问题，但我在 auth->login() 方面遇到了问题，经过长时间的调试，我想分享我的发现。这对我来说很可能是愚蠢的事情，但请耐心等待......

模型“Cliente”(相关部分):

<?php
App::uses('AppModel', 'Model');
App::uses('BrValidation', 'Localized.Validation');
App::uses('AuthComponent', 'Controller/Component');
App::uses('SimplePasswordHasher', 'Controller/Component/Auth');

/**  
* Cliente Model
*
*/
class Cliente extends AppModel {
    public $primaryKey = 'idcliente';

public function beforeSave($options = array()) {
    if (isset($this->data[$this->alias]['password'])) {
        $passwordHasher = new SimplePasswordHasher();
        $this->data[$this->alias]['password'] = $passwordHasher->hash(
            $this->data[$this->alias]['password']
        );
    }
    return true;
}
}

ClientesController(相关部分):

<?php
App::uses('AppController', 'Controller');
App::uses('CakeEmail', 'Network/Email');

/**
* Clientes Controller
*
*/
class ClientesController extends AppController {

public $components = array('Paginator','RequestHandler');

/** beforeFilter
*  Configura partes do sistema que podem ser acessadas sem login
*/ 

public function beforeFilter() {
    //parent::beforeFilter();
    //$this->Auth->allow();
}
public function login() {

    //if already logged-in, redirect
    if($this->Session->check('Auth.Cliente')){
        $this->redirect(array('action' => 'index'));      
    }

    // if we get the post information, try to authenticate
    if ($this->request->is('post')) {
        CakeLog::write('debug',"POS IF POST\n");
        if ($this->Auth->login()) {
            CakeLog::write('debug',"True no LOGIN\n");
            $this->Session->setFlash(__('Welcome, '. $this->Auth->user('Cliente.username')));
            $this->redirect($this->Auth->redirectUrl());
        } else {
            $this->Session->setFlash(__('Invalid username or password'));
        }
    } 
}

AppController(相关部分) 类 AppController 扩展了 Controller {

public $components = array(
    'DebugKit.Toolbar',
    'Session',
    'Auth' => array(
        'loginAction' => array('controller' => 'clientes', 'action' => 'login'),
        'loginRedirect' => array('controller' => 'clientes', 'action' => 'poslogin'),
        'logoutRedirect' => array('controller' => 'clientes', 'action' => 'login'),
        'authError' => 'Você precisa se logar para ver esta página.',
        'loginError' => 'Usuário ou senha inválidos.  Tente de novo.',
        'authenticate' => array(
        'Form' => array(
            'userModel' => 'Cliente',
            'fields' => array ( 'username' => 'username', 'password' => 'password')
        )
    )

    ));

无需发布关联的 View ，因为表单会收集信息并按预期发布。

就目前情况而言，$this->Auth->login() 将始终返回 TRUE，无论 POSTed 数据如何。用户是否存在以及密码是否正确并不重要。

Cake 正在查找正确的表和正确的用户。我已经打开了MySQL General Log以确保实际查询是正确的，所以这里没有问题。

然后我转向实际的 Cake 代码并发现以下内容。我们感兴趣的代码位于$ROOT/lib/Cake/Controller/Component/AuthComponent.php ，第 595-607 行(login() 函数)，如下:

public function login($user = null) {
    $this->_setDefaults();

    if (empty($user)) {
        $user = $this->identify($this->request, $this->response);
    }
    if ($user) {
        $this->Session->renew();
        $this->Session->write(self::$sessionKey, $user);
    }
    CakeLog::write('debug',"LOGGEDIN:". $this->loggedIn());
    return $this->loggedIn();
}

由于我们不传递任何参数，Cake 将尝试 identify用户(第 599 行)然后将返回其 loggedIn当前状态。 loggedIn只是 user() 的 bool 结果正如我们在第 820-822 行中看到的那样，让我们​​看一下 user() (第 648-658 行)。

public static function user($key = null) {
    if (!empty(self::$_user)) {
        $user = self::$_user;
    } elseif (self::$sessionKey && CakeSession::check(self::$sessionKey)) {
        $user = CakeSession::read(self::$sessionKey);
    } else {
        CakeLog::write('debug',"return null");
        return null;
    }
    if ($key === null) {
        ob_start();
        var_dump($user);
        $d=ob_get_contents();
        ob_clean();
        CakeLog::write('debug',"return user: " . $d);
        return $user;
    }
    CakeLog::write('debug',"return hash" . Hash::get($user,$key));
    return Hash::get($user, $key);
}

自从我们调用 user()没有参数，$key将为 NULL，代码将简单地 return $user而且——至少对我来说——这就是问题所在。如果我var_dump($user)如上所述，我得到一个空数组，但我得到一些“TRUE”并且登录过程“成功”(当然是错误的)。

2014-05-25 00:24:04 Debug: return user: array(1) {
  ["Cliente"]=>
  array(3) {
    ["idcliente"]=>
    NULL
    ["username"]=>
    NULL
    ["password"]=>
    NULL
 }
}

我正在使用 Cake 2.4.1，但我已经检查了 2.4.10 之前的所有 2.4 版本，并且代码没有更改，因此这将在整个 Cake 2.4 中发生。不过我还没有查过其他小版本。我在 OSX 上运行 PHP 5.5.3，但这听起来与我不太相关。

我还没有阅读任何提出此问题的变更日志，并且 SO 中有一些关于此问题的未解答问题。

我不想在不先共享的情况下更改核心代码，并且不确定 SO 是否是发布此内容的“正确”位置，很可能是我只是忽略了某些内容。

如果有人能够阐明一些观点并提出一些想法，我们将不胜感激。

谢谢!

Answer 1

我已经成功缩小了问题范围，虽然它似乎不是核心代码中的错误，但它似乎确实符合需要进一步调查的行为。

假设您有一个“干净”的浏览器 session ，没有 session COOKIES。在这种情况下，代码将按预期工作。错误的用户和/或密码将被拒绝。

但是，一旦您登录，Cake 将在浏览器中处理一个 session cookie (CAKEPHP)，除其他信息外，该 cookie 还会将该 session 链接到已登录的用户。

如果您尝试再次登录(“在当前登录之上”)，$this->Auth->login()(不带参数)无论如何都会产生 TRUE。如果您提供了错误的用户/密码，它将坚持当前有效的用户/密码。如果您提供有效的用户/密码，它将更改为新用户。

除非您注销，或者 session cookie 过期(或者您手动删除它，例如)，否则您是否使用以下命令调用 $this->Auth->login() 并不重要或不带参数。

虽然从“页面流”/逻辑角度来看，这似乎是有道理的(为什么在登录后会再次调用“登录”？)，但从用户行为的角度来看，这可能会导致(如此处所注意到的!)奇怪的问题。

假设您登录，执行您想做的任何操作，然后离开您的网络应用程序而不注销。在 Cake 的 session Cookie 期间，您将保持登录状态。如果您与其他人共享浏览器和应用程序，此人可能会意外地通过登录但输入错误的密码来操作您的帐户，在这种情况下，如上所述，login() 将返回 TRUE ，但不会更改用户。更不用说潜在的恶意使用。

无论如何...在登录之前强制执行 $this->Auth->logout() 以避免潜在的应用不当行为可能是一个好习惯。

由于这种方法似乎贯穿 Cake 2.x，我希望我的问题也能澄清其他人。