From bf270745f0dc540aa72f9bbd7cc2f9df9c91cbd4 Mon Sep 17 00:00:00 2001
From: James Cole <thegrumpydictator@gmail.com>
Date: Sat, 9 Sep 2017 21:57:45 +0200
Subject: [PATCH] Separate commit for error handler.

---
 app/Exceptions/Handler.php | 109 ++++++-------------------------------
 1 file changed, 17 insertions(+), 92 deletions(-)

diff --git a/app/Exceptions/Handler.php b/app/Exceptions/Handler.php
index 14cb6b0df5..c17d9d7ad2 100644
--- a/app/Exceptions/Handler.php
+++ b/app/Exceptions/Handler.php
@@ -1,128 +1,53 @@
 <?php
-/**
- * Handler.php
- * Copyright (C) 2016 thegrumpydictator@gmail.com
- *
- * This software may be modified and distributed under the terms of the
- * Creative Commons Attribution-ShareAlike 4.0 International License.
- *
- * See the LICENSE file for details.
- */
-
-declare(strict_types=1);
 
 namespace FireflyIII\Exceptions;
 
-use ErrorException;
 use Exception;
-use FireflyIII\Jobs\MailError;
-use Illuminate\Auth\Access\AuthorizationException;
-use Illuminate\Auth\AuthenticationException;
-use Illuminate\Database\Eloquent\ModelNotFoundException;
 use Illuminate\Foundation\Exceptions\Handler as ExceptionHandler;
-use Illuminate\Session\TokenMismatchException;
-use Illuminate\Validation\ValidationException as ValException;
-use Request;
-use Symfony\Component\HttpKernel\Exception\HttpException;
 
-/**
- * Class Handler
- *
- * @package FireflyIII\Exceptions
- */
 class Handler extends ExceptionHandler
 {
     /**
-     * A list of the exception types that should not be reported.
+     * A list of the exception types that are not reported.
      *
      * @var array
      */
-    protected $dontReport
-        = [
-            AuthenticationException::class,
-            AuthorizationException::class,
-            HttpException::class,
-            ModelNotFoundException::class,
-            TokenMismatchException::class,
-            ValException::class,
-        ];
+    protected $dontReport = [
+        //
+    ];
 
     /**
-     * Render an exception into an HTTP response.
+     * A list of the inputs that are never flashed for validation exceptions.
      *
-     * @param  \Illuminate\Http\Request $request
-     * @param  \Exception               $exception
-     *
-     * @return \Illuminate\Http\Response
+     * @var array
      */
-    public function render($request, Exception $exception)
-    {
-        if ($exception instanceof FireflyException || $exception instanceof ErrorException) {
-
-            $isDebug = env('APP_DEBUG', false);
-
-            return response()->view('errors.FireflyException', ['exception' => $exception, 'debug' => $isDebug], 500);
-        }
-
-        return parent::render($request, $exception);
-    }
-
+    protected $dontFlash = [
+        'password',
+        'password_confirmation',
+    ];
 
     /**
      * Report or log an exception.
      *
      * This is a great spot to send exceptions to Sentry, Bugsnag, etc.
      *
-     * @param  Exception $exception
-     * @SuppressWarnings(PHPMD.CyclomaticComplexity) // it's exactly five.
-     *
+     * @param  \Exception  $exception
      * @return void
      */
     public function report(Exception $exception)
     {
-        $doMailError = env('SEND_ERROR_MESSAGE', true);
-        if (($exception instanceof FireflyException || $exception instanceof ErrorException) && $doMailError) {
-            $userData = [
-                'id'    => 0,
-                'email' => 'unknown@example.com',
-            ];
-            if (auth()->check()) {
-                $userData['id']    = auth()->user()->id;
-                $userData['email'] = auth()->user()->email;
-            }
-            $data = [
-                'class'        => get_class($exception),
-                'errorMessage' => $exception->getMessage(),
-                'time'         => date('r'),
-                'stackTrace'   => $exception->getTraceAsString(),
-                'file'         => $exception->getFile(),
-                'line'         => $exception->getLine(),
-                'code'         => $exception->getCode(),
-                'version'      => config('firefly.version'),
-            ];
-
-            // create job that will mail.
-            $ipAddress = Request::ip() ?? '0.0.0.0';
-            $job       = new MailError($userData, env('SITE_OWNER', ''), $ipAddress, $data);
-            dispatch($job);
-        }
-
         parent::report($exception);
     }
 
     /**
-     * Convert an authentication exception into an unauthenticated response.
+     * Render an exception into an HTTP response.
      *
-     * @param $request
-     *
-     * @return \Illuminate\Http\JsonResponse|\Illuminate\Http\RedirectResponse
+     * @param  \Illuminate\Http\Request  $request
+     * @param  \Exception  $exception
+     * @return \Illuminate\Http\Response
      */
-    protected function unauthenticated($request)
+    public function render($request, Exception $exception)
     {
-        if ($request->expectsJson()) {
-            return response()->json(['error' => 'Unauthenticated.'], 401);
-        }
-
-        return redirect()->guest('login');
+        return parent::render($request, $exception);
     }
 }