Log the error output for API
This commit is contained in:
parent
af68dbed8f
commit
c8a73fa608
3 changed files with 11 additions and 9 deletions
|
@ -62,15 +62,15 @@ class APISecretToken extends Authorization
|
|||
public function authenticate(Request $request, Route $route)
|
||||
{
|
||||
if (!$request->bearerToken() || empty($request->bearerToken())) {
|
||||
APILogService::log($request);
|
||||
throw new UnauthorizedHttpException('The authentication header was missing or malformed');
|
||||
APILogService::log($request, 'The authentication header was missing or malformed.');
|
||||
throw new UnauthorizedHttpException('The authentication header was missing or malformed.');
|
||||
}
|
||||
|
||||
list($public, $hashed) = explode('.', $request->bearerToken());
|
||||
|
||||
$key = APIKey::where('public', $public)->first();
|
||||
if (!$key) {
|
||||
APILogService::log($request);
|
||||
APILogService::log($request, 'Invalid API Key.');
|
||||
throw new AccessDeniedHttpException('Invalid API Key.');
|
||||
}
|
||||
|
||||
|
@ -85,7 +85,7 @@ class APISecretToken extends Authorization
|
|||
}
|
||||
}
|
||||
if (!$inRange) {
|
||||
APILogService::log($request);
|
||||
APILogService::log($request, 'This IP address <' . $request->ip() . '> does not have permission to use this API key.');
|
||||
throw new AccessDeniedHttpException('This IP address <' . $request->ip() . '> does not have permission to use this API key.');
|
||||
}
|
||||
}
|
||||
|
@ -98,7 +98,7 @@ class APISecretToken extends Authorization
|
|||
}
|
||||
|
||||
if (!$this->permissionAllowed) {
|
||||
APILogService::log($request);
|
||||
APILogService::log($request, 'You do not have permission to access this resource.');
|
||||
throw new AccessDeniedHttpException('You do not have permission to access this resource.');
|
||||
}
|
||||
}
|
||||
|
@ -106,18 +106,18 @@ class APISecretToken extends Authorization
|
|||
try {
|
||||
$decrypted = Crypt::decrypt($key->secret);
|
||||
} catch (\Illuminate\Contracts\Encryption\DecryptException $ex) {
|
||||
APILogService::log($request);
|
||||
APILogService::log($request, 'There was an error while attempting to check your secret key.');
|
||||
throw new HttpException('There was an error while attempting to check your secret key.');
|
||||
}
|
||||
|
||||
$this->url = urldecode($request->fullUrl());
|
||||
if($this->_generateHMAC($request->getContent(), $decrypted) !== base64_decode($hashed)) {
|
||||
APILogService::log($request);
|
||||
APILogService::log($request, 'The hashed body was not valid. Potential modification of contents in route.');
|
||||
throw new BadRequestHttpException('The hashed body was not valid. Potential modification of contents in route.');
|
||||
}
|
||||
|
||||
// Log the Route Access
|
||||
APILogService::log($request, true);
|
||||
APILogService::log($request, null, true);
|
||||
return true;
|
||||
|
||||
}
|
||||
|
|
|
@ -36,7 +36,7 @@ class APILogService
|
|||
//
|
||||
}
|
||||
|
||||
public static function log(Request $request, $authorized = false)
|
||||
public static function log(Request $request, $error = null, $authorized = false)
|
||||
{
|
||||
if ($request->bearerToken() && !empty($request->bearerToken())) {
|
||||
list($public, $hashed) = explode('.', $request->bearerToken());
|
||||
|
@ -47,6 +47,7 @@ class APILogService
|
|||
try {
|
||||
$log = APILog::create([
|
||||
'authorized' => $authorized,
|
||||
'error' => $error,
|
||||
'key' => $public,
|
||||
'method' => $request->method(),
|
||||
'route' => $request->fullUrl(),
|
||||
|
|
|
@ -16,6 +16,7 @@ class BuildApiLogTable extends Migration
|
|||
Schema::create('api_logs', function (Blueprint $table) {
|
||||
$table->increments('id');
|
||||
$table->boolean('authorized');
|
||||
$table->text('error')->nullable();
|
||||
$table->char('key', 16)->nullable();
|
||||
$table->char('method', 6);
|
||||
$table->text('route');
|
||||
|
|
Loading…
Reference in a new issue