fix: http exception let the global exception handler handle it

Fixes #22

Author: thetutlage

npm-audit.html

@@ -55,7 +55,7 @@ <h5 class="card-title">
                 <div class="card">
                     <div class="card-body">
                         <h5 class="card-title">
-                            August 27th 2020, 3:46:36 pm
+                            August 27th 2020, 3:51:23 pm
                         </h5>
                         <p class="card-text">Last updated</p>
                     </div>

src/Exceptions/HttpException.ts

@@ -10,7 +10,6 @@
 /// <reference path="../../adonis-typings/index.ts" />
 
 import { Exception, interpolate } from '@poppinss/utils'
-import { HttpContextContract } from '@ioc:Adonis/Core/HttpContext'
 
 import { E_HTTP_EXCEPTION } from '../../exceptions.json'
 
@@ -37,12 +36,4 @@ export class HttpException extends Exception {
 		error.body = error.message
 		return error
 	}
-
-	/**
-	 * Handle itself by making the response. This only works when using the
-	 * base exception handler shipped by AdonisJs
-	 */
-	public handle(error: HttpException, ctx: HttpContextContract) {
-		ctx.response.status(error.status).send(error.body)
-	}
 }

src/Response/index.ts

@@ -41,6 +41,16 @@ import { CookieSerializer } from '../Cookie/Serializer'
 import { HttpException } from '../Exceptions/HttpException'
 import { E_CANNOT_SERIALIZE_RESPONSE_BODY } from '../../exceptions.json'
 
+class AbortException extends HttpException {
+	/**
+	 * Handle itself by making the response. This only works when using the
+	 * base exception handler shipped by AdonisJs
+	 */
+	public handle(error: HttpException, ctx: HttpContextContract) {
+		ctx.response.status(error.status).send(error.body)
+	}
+}
+
 /**
  * The response is a wrapper over [ServerResponse](https://nodejs.org/api/http.html#http_class_http_serverresponse)
  * streamlining the process of writing response body and automatically setting up appropriate headers.
@@ -809,7 +819,7 @@ export class Response extends Macroable implements ResponseContract {
 	 * used when status is not defined
 	 */
 	public abort(body: any, status?: number): never {
-		throw HttpException.invoke(body, status || 400)
+		throw AbortException.invoke(body, status || 400)
 	}
 
 	/**