记录一个因为使用不规范引发的debug开启后报错问题

2020-05-18

字数统计: 1.2k | 阅读时长≈ 4 分钟

我想不止我一个人在使用thinkPHP的时候出现过很多诡异的情况，也不止我一个人在打开和关闭框架中的DEBUG时得到2种运行结果。这次就较一下真，看下到底是框架垃圾还是我们使用过程中的确存在不规范（人垃圾）

背景

因为事业环境因素需要统一过滤项目组中各个业务线的搜索词是否合法。采用的解决方案如下

用Go的生态 load 一颗 trie树到内存
PHP 走 RPC 调用启动的这个Go服务
PHP 提供一个短连接的内网RPC服务供各个业务线使用

疑问：

为什么要用trie树

trie树 + AC自动机是目前比较高效的违禁词过滤方法
为什么不直接调用Go服务而要过一层PHP

想在这层PHP上做熔断报警机制，业务层并发量大PHP的curl库超时最少1秒，在并发量比较高的场景1秒的等待开销太大。

技术选型

RPC协议
小马哥的Hprose

项目框架
thinkPHP3.1 (历史原因)

常驻监控
supervisor

问题

在实际操作期间，其实蛮顺利的

GO常驻服务&&接口暴露
服务层PHP调用Go接口
应用层PHP调用PHP提供的服务接口

只是再最后一步出现了一个问题我发现应用层的PHP调用服务层的掉不通，而且会抛出错误。经过一些调试发现最诡异的是把thinkPHP的APP_DEBUG关掉接能正常调用

解决

首先想到的是各种语法层面的问题

发现Hprose的方法发布核心函数addFunction 是这样写的

public function addFunction($func, $alias = '', array $options = array()) {
       if (!is_callable($func)) {
           throw new Exception('Argument func must be callable.');
       }
       if (is_array($alias) && empty($options)) {
           $options = $alias;
           $alias = '';
       }
       if (empty($alias)) {
           if (is_string($func)) {
               $alias = $func;
           }
		...

从is_callable可以得出作者想让我们传入的是一个回调函数，这里为了排除是魔改过后的框架有类调用找不到的情况，传入一个匿名函数个人感觉比较稳妥而且没什么问题（排除法慢慢来）代码如下

function xxx()
{
    $server = new \Hprose\Http\Server();

    $server->addFunction(function ($keyword)  {
        try {
            return $this->client->Validate($keyword);
        }catch (Exception $e) {
            return $this->error($keyword);
        }

    }, "validate");
    $server->start();
}

尝试了这种方法后继续掉用，发现问题还是没有得到解决但是报错信息变成了

Exception: Wrong Response: 
Rm2{s5"state"s5"false"s7"keyword" in /Users/zhangjunjie/code/testdocker/phpdocker/www/blog/vendor/hprose/hprose/src/Hprose/Client.php:351
Stack trace:

我们定位到 Client.php:351 行