Merge branch 'feature/new_wms' into 'master'

签名验证(带多余请求)

See merge request composer/php-wms-client!1
This commit is contained in:
郑翀 2020-07-17 11:36:26 +08:00
commit 5dc67b272f

View File

@ -611,15 +611,8 @@ class QimenWms extends BaseWms implements WmsInterface
if (empty($params)) { if (empty($params)) {
return false; return false;
} }
$query = [ $query = $params;
'method' => $params['method'], unset($query['body'], $query['sign']);
'timestamp' => $params['timestamp'],
'format' => $params['format'],
'app_key' => $params['app_key'],
'v' => $params['v'],
'sign_method' => $params['sign_method'],
'customerId' => $params['customerId'],
];
$sign = $this->createSign($query, $params['body']); $sign = $this->createSign($query, $params['body']);
if (!hash_equals((string) $sign, (string) $params['sign'])) { if (!hash_equals((string) $sign, (string) $params['sign'])) {
return false; return false;