使用AJAX调用PHP类方法

Question

我想出了以下代码来通过PHP类中的AJAX调用方法：

PHP：

class Ajax extends Controller {

    private $class;
    private $method;
    private $params;

    function __construct()
    {
        $this->params = $_POST; // Call params
        $call = explode('->', $this->params['call']);
        $this->class = new $call[0]; // e.g. controller->method
        $this->method = $call[1];
        array_shift($this->params);
        $this->parse();
    }

    public function index()
    {
        //Dummy
    }

    public function parse()
    {
        $r = '';
        $r = call_user_func_array(array($this->class, $this->method), $this->params);
        echo $r;
    }
}

客户：

function creditCheck2(id)
{
    $.post(ROOT + 'Ajax', {call: 'Record->creditState', id: id, enquiryid: enquiryId}, function(data) {
        alert(data)
    }, 'json')
}

它看起来很棒，但是它是否安全并且会更好？

---

仅供参考，我在代码中添加了答案建议的更改：

class Call extends Controller {

  private $class;
  private $method;
  private $params;
  private $authClasses = array(
      'Gallery'
  );

  function __construct()
  {
    $this->params = $_POST; // Call params
    $call = explode('->', $this->params['call']);

    if(!in_array($call[0], $this->authClasses))
    {
      die();
    }
    $this->class = new $call[0]; // e.g. controller->method
    $this->method = $call[1];
    unset($this->params['call']);
    $this->parse();
  }

  public function parse()
  {
    $r = '';
    $param = array();

    // Params in any order...
    $mRef = new ReflectionMethod($this->class, $this->method);
    foreach($mRef->getParameters() as $p) {
      $param[$p->name] = $this->params[$p->name];
    }
    $this->params = $param;

    if($r = @call_user_func_array(array($this->class, $this->method), $this->params))
    {
      echo $r;
    }
    else {

    }
  }

}

Answer 1

小问题

最好在array_shift($this->params)不必要地假定params数组中的第一项始终为call 。 这是不正确的，它与您之前做的直接访问$this->params['call'] 。 应该用简单的unset($this->params['call'])替换array_shift 。

更大的问题

还有一个问题是，params数组中的值的顺序必须与您尝试调用的方法的签名中的参数的顺序匹配。 我认为不能保证顺序与AJAX请求中参数的顺序相同，所以这是一个理论问题。

非常大的问题

更重要的是，这种处理方式会迫使 AJAX代码的作者匹配您要调用的方法签名中的参数顺序。 这引入了可怕的耦合水平，并且是一个主要问题。 更糟糕的是，错误地更改参数的顺序将不明显。 考虑：

public function bankTransfer($fromAccount, $toAccount, $amount);

$.post(ROOT + 'Ajax', {
    call: 'Bank->bankTransfer', 
    from: "sender",
    to: "recipient",
    amount: 42
}, function(data) { ... });

这会起作用。 但是如果你这样做

$.post(ROOT + 'Ajax', {
    call: 'Bank->bankTransfer', 
    to: "recipient", // swapped the order of
    from: "sender",  // these two lines
    amount: 42
}, function(data) { ... });

您将得到与预期相反的结果。 我相信这很明显是很明显的。

要解决该问题，您将必须使用反射将$this->params中的数组键与所调用方法的参数的形式名称进行匹配。

安全

最后，此代码是不安全的，因为任何人都可以发出请求，以指导您的代码调用具有适当参数的任何类的任何方法，甚至是不应从Web环境访问的方法。

这是另一个严重的问题，除非您对调度逻辑引入某种类型的过滤，否则无法真正解决。

Answer 2

它看起来很棒，但是它是否安全并且会更好？

您是使用自己的框架还是其他框架？ 我相信，如果攻击者知道您的框架中可能包含的内容，则根本不安全。 例如：您的框架中有数据库类，攻击者可以执行以下操作：

{call: 'Database->execute', sql: 'SELECT * FROM information_schema.`tables`'}

筛选

您可以限制允许用户访问的类的数量。 例如：

if (!in_array($this->class, array('Record', 'Hello'))) {
    die();
}

反射

这是我刚刚学习的反射示例（感谢@Jon供参考）。 这解决了从PHP函数以不同顺序传递参数的问题。

class Email
{
    public function send($from, $to, $msg) {
        return "Send $from to $to: $msg";
    }
}

$rawParam = array('msg' => 'Hello World', 
               'to' => 'to@gmail.com', 
               'from' => 'from@gmail.com');
$param = array();

// Rearrange
$methodRef = new ReflectionMethod('Email', 'send');
foreach($methodRef->getParameters() as $p) {
    $param[$p->name] = $rawParam[$p->name];
}

var_dump($rawParam);
var_dump($param);