Created
March 2, 2016 09:43
-
-
Save argentinaluiz/2cc374e74a958798fb46 to your computer and use it in GitHub Desktop.
Solution to the conflict between oauth2-server-laravel and laravel-cors in return response header in oauth error
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
<?php | |
namespace App\Exceptions; | |
use Asm89\Stack\CorsService; | |
use Exception; | |
use Illuminate\Database\Eloquent\ModelNotFoundException; | |
use League\OAuth2\Server\Exception\OAuthException; | |
use Psr\Log\LoggerInterface; | |
use Symfony\Component\HttpKernel\Exception\HttpException; | |
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException; | |
use Illuminate\Foundation\Exceptions\Handler as ExceptionHandler; | |
class Handler extends ExceptionHandler | |
{ | |
/** | |
* @var CorsService | |
*/ | |
private $corsService; | |
public function __construct(LoggerInterface $log, CorsService $corsService) | |
{ | |
parent::__construct($log); | |
$this->corsService = $corsService; | |
} | |
/** | |
* A list of the exception types that should not be reported. | |
* | |
* @var array | |
*/ | |
protected $dontReport = [ | |
HttpException::class, | |
ModelNotFoundException::class, | |
]; | |
/** | |
* Report or log an exception. | |
* | |
* This is a great spot to send exceptions to Sentry, Bugsnag, etc. | |
* | |
* @param \Exception $e | |
* @return void | |
*/ | |
public function report(Exception $e) | |
{ | |
return parent::report($e); | |
} | |
/** | |
* Render an exception into an HTTP response. | |
* | |
* @param \Illuminate\Http\Request $request | |
* @param \Exception $e | |
* @return \Illuminate\Http\Response | |
*/ | |
public function render($request, Exception $e) | |
{ | |
if ($e instanceof ModelNotFoundException) { | |
$e = new NotFoundHttpException($e->getMessage(), $e); | |
} elseif ($e instanceof OAuthException) { | |
$response = response()->json([ | |
'error' => $e->errorType, | |
'error_description' => $e->getMessage() | |
], $e->httpStatusCode, $e->getHttpHeaders()); | |
return $this->corsService->addActualRequestHeaders($response, $request); | |
} else { | |
return parent::render($request, $e); | |
} | |
} | |
} |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Needs comments
\LucaDegasperi\OAuth2Server\Middleware\OAuthExceptionHandlerMiddleware::class
in Kernel.php