blax-software
/
laravel-websockets
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix app secret. 

Only add the md5 of the body, if it's present
This commit is contained in:
Marcel Pociot 2018-12-03 10:08:25 +01:00
parent 1bf844d631
commit 8f4dfc0e1e
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/HttpApi/Controllers/Controller.php
View File

@ -86,16 +86,20 @@ abstract class Controller implements HttpServerInterface
protected function ensureValidSignature(Request $request)
{
$bodyMd5 = md5($request->getContent());
$signature =
"{$request->getMethod()}\n/{$request->path()}\n" .
"auth_key={$request->get('auth_key')}" .
"&auth_timestamp={$request->get('auth_timestamp')}" .
"&auth_version={$request->get('auth_version')}" .
"&body_md5={$bodyMd5}";
"&auth_version={$request->get('auth_version')}";
$authSignature = hash_hmac('sha256', $signature, App::findById($request->get('appId'))->appSecret);
if ($request->getContent() !== '') {
$bodyMd5 = md5($request->getContent());
$signature .= "&body_md5={$bodyMd5}";
}
$authSignature = hash_hmac('sha256', $signature, App::findById($request->get('appId'))->secret);
if ($authSignature !== $request->get('auth_signature')) {
throw new HttpException(401, 'Invalid auth signature provided.');