Como habrás visto en el método que estás sobreescribiendo, Illuminate\Foundation\Exceptions\Handler::render
, en él se "rutea" la excepción acorde a sus atributos. Hay muchos casos en donde tu idea funciona, dado que lo indispensable es poder identificar el statusCode de una excepción. En los casos que no, estás invocando al método getStatusCode
sin tener en cuenta la contravarianza. El método acepta Throwable
de cualquier tipo, no necesariamente del subtipo que tiene statusCode.
Veamos los casos favorables:
Si la excepción implementa la interfaz Symfony\Component\HttpKernel\Exception\HttpExceptionInterface
entonces tiene un accesor directo al statusCode
$httpException->getStatusCode();
Si una excepción es de tipo Illuminate\Http\Exceptions\HttpResponseException
entonces tiene una propiedad response
con el accesor getResponse
. Por transitividad, tiene un statusCode:
$httpResponseException->getResponse()->getStatusCode();
Otros errores específicos de Laravel se pueden mapear a excepciones de la familia HttpException de Symfony de manera directa:
- ModelNotFoundException -> NotFoundHttpException
- RecordNotFoundException -> NotFoundHttpException
- AuthorizationException -> AccessDeniedHttpException
(de todos modos éstos son casos particulares de HttpExceptionInterface
así que están cubiertos por la primera condición)
Hay otros casos en donde la excepción implementa la interfaz Illuminate\Contracts\Support\Responsable
, que no tiene específicamente una respuesta o un statusCode pero puede convertirse (coercionarse) a una respuesta con el método toResponse
. Por transitividad, tiene un statusCode.
$responsable->toResponse()->getStatusCode()
Finalmente hay casos particulares en donde la excepción por sí misma no te puede decir su statuscode pero la lógica de dominio sabe que (por ejemplo) una excepción AuthenticationException
arrojada por un middleware y sus guards equivaldría a un status code 401 si acaso fuera pertinente expresarla como respuesta (que no siempre es así).
Para todo el resto de las docenas de excepciones posibles no hay una equivalencia con un determinado statusCode. A grandes rasgos:
public function obtenerStatusCode(\Throwable $e) {
if($e instanceof \Symfony\Component\HttpKernel\Exception\HttpExceptionInterface) {
return $e->getStatusCode();
}
if($e instanceof \Illuminate\Http\Exceptions\HttpResponseException) {
return $e->getResponse()->getStatusCode();
}
if($e instanceof \Illuminate\Contracts\Support\Responsable) {
return $e->toResponse()->getStatusCode();
}
if($e instanceof \Illuminate\Auth\AuthenticationException) {
return 401;
}
// y si entras aquí es porque te toca definir qué se hace con otros casos
}
Con respecto a tu pregunta
¿a dónde se dirige entonces? ¿las vistas 404 y 503 se muestran correctamente?
Tú tienes definido que si el statusCode es 404 se dibuje esa vista. Significa que la excepción es (posiblemente) del tipo NotFoundHttpException
de Symfony, que a su vez puede haberse arrojado como consecuencia de excepciones de Laravel como ModelNotFoundException
o RelationNotFoundException
o simplemente una ruta inexistente.
De la misma manera, tienes definida la vista para las excepciones cuyo statusCode es 503, o sea viene del tipo ServiceUnavailableHttpException
de Symfony que a su vez puede venir del tipo MaintenanceModeException
de Laravel.
De todos los mentados tipos, no hay ninguno cuyo statusCode puedas obtener directamente que te vaya a devolver 500, porque el error 500 es el fallback para todas las excepciones que no tiene objeto clasificar o simplemente no capturaste correctamente.
Como desde ya sabes que lo que te llega es de tipo ErrorException
, lo más probable es que venga de Illuminate\Foundation\Bootstrap\HandleExceptions::handleError
y puede originarse de cualquier error de PHP. Lo más sano es que manejes esa condición definiendo:
if($e instanceof \ErrorException) {
return 500;
}
Si es que eso tiene sentido en tu flujo.
Edit: Otro punto de vista
La respuesta se basa en tu pregunta, en lo que quieres obtener y espero más te sirva.
Sin embargo hay otra escuela de pensamiento que me hace mucho sentido, respecto a los headers:
- headers 200 y 300: todo bien
- headers 4xx : es tu culpa
- headers 5xx : es nuestra culpa
En el sentido que la familia de statusCode 4xx indica un error del usuario, por ejemplo cuando pide una ruta que no existe o manda a persistir una entidad con un ID incompatible, o cuando pide una ruta sin estar autenticado. Es su culpa.
Un error 404 cuando pide una entidad en una ruta correcta pero el modelo no existe es en realidad un error engañoso. El usuario no ha cometido ningún error.
De todos modos si lo importante es mostrarle un mensaje con feedback es mejor apelar al 404 que nada.