Server - Personalizada la pagina de error y agregado nueva informacio…

…n de debug
TheMauzoTeam · May 16, 2020 · ebb2b41 · ebb2b41
1 parent 3d8c0ba
commit ebb2b41
Show file tree

Hide file tree

Showing 6 changed files with 51 additions and 8 deletions.
diff --git a/pom.xml b/pom.xml
@@ -78,6 +78,11 @@
       <artifactId>jjwt-jackson</artifactId>
       <version>${jwt.version}</version>
     </dependency>
+    <dependency>
+      <groupId>org.springframework.boot</groupId>
+      <artifactId>spring-boot-starter-thymeleaf</artifactId>
+      <version>${springboot.version}</version>
+    </dependency>
 
     <!-- Dependencias exclusivas en los entornos de test -->
     <dependency>

diff --git a/src/main/java/io/Mauzo/Server/Controllers/SalesCtrl.java b/src/main/java/io/Mauzo/Server/Controllers/SalesCtrl.java
@@ -58,6 +58,7 @@ public Response getSalesMethod(@Context final HttpServletRequest req) {
                     jsonObj.add("userId", saleAux.getUserId());
                     jsonObj.add("prodId", saleAux.getProdId());
 
+                    // Capturamos posible null procedente de la BBDD.
                     try {
                         jsonObj.add("discId", saleAux.getDiscId());
                     } catch (NullPointerException e) {
@@ -136,6 +137,7 @@ public Response getSaleMethod(@Context final HttpServletRequest req, @PathParam(
                 jsonResponse.add("userId", saleAux.getUserId());
                 jsonResponse.add("prodId", saleAux.getProdId());
 
+                // Capturamos posible null procedente de la BBDD.
                 try {
                     jsonResponse.add("discId", saleAux.getDiscId());
                 } catch (NullPointerException e) {

diff --git a/src/main/java/io/Mauzo/Server/Controllers/UsersCtrl.java b/src/main/java/io/Mauzo/Server/Controllers/UsersCtrl.java
@@ -72,6 +72,7 @@ public Response getLists(@Context final HttpServletRequest req) {
                     jsonObj.add("email", user.getEmail());
                     jsonObj.add("isAdmin", user.isAdmin());
 
+                    // Capturamos posible null procedente de la BBDD.
                     try {
                         jsonObj.add("userPic", ServerUtils.byteArrayToBase64(ServerUtils.imageToByteArray(user.getUserPic(), "png")));
                     } catch (NullPointerException e) {
@@ -181,6 +182,7 @@ public Response getUser(@Context final HttpServletRequest req, @PathParam("param
                 jsonResponse.add("email", user.getEmail());
                 jsonResponse.add("isAdmin", user.isAdmin());
 
+                // Capturamos posible null procedente de la BBDD.
                 try {
                     jsonResponse.add("userPic", ServerUtils.byteArrayToBase64(ServerUtils.imageToByteArray(user.getUserPic(), "png")));
                 } catch (NullPointerException e) {

diff --git a/src/main/java/io/Mauzo/Server/ServerUtils.java b/src/main/java/io/Mauzo/Server/ServerUtils.java
@@ -1,5 +1,6 @@
 package io.Mauzo.Server;
 
+import java.util.Arrays;
 // Paquetes del framework estandar de java
 import java.util.Base64;
 import java.util.List;
@@ -80,7 +81,7 @@ public static Response genericMethod(HttpServletRequest req, Integer paramId, St
 
             // Informacion necesaria en procesos de debug.
             ServerApp.getLoggerSystem().debug(e.getMessage());
-            ServerApp.getLoggerSystem().debug(e.getStackTrace().toString());
+            ServerApp.getLoggerSystem().debug(Arrays.toString(e.getStackTrace()));
 
             response = Response.serverError();
         } catch (Exception e) {
@@ -89,7 +90,7 @@ public static Response genericMethod(HttpServletRequest req, Integer paramId, St
 
             // Informacion necesaria en procesos de debug.
             ServerApp.getLoggerSystem().debug(e.getMessage());
-            ServerApp.getLoggerSystem().debug(e.getStackTrace().toString());
+            ServerApp.getLoggerSystem().debug(Arrays.toString(e.getStackTrace()));
 
             response = Response.serverError();
         }
@@ -131,14 +132,14 @@ public static Response genericUserMethod(HttpServletRequest req, Integer paramId
                 // Detectamos errores en la SQL
                 ServerApp.getLoggerSystem().warn("Error en procesar la consulta SQL: " + e.toString());
                 ServerApp.getLoggerSystem().debug(e.getMessage());
-                ServerApp.getLoggerSystem().debug(e.getStackTrace().toString());
+                ServerApp.getLoggerSystem().debug(Arrays.toString(e.getStackTrace()));
 
                 response = Response.serverError();
             } catch (Exception e) {
                 // En caso de existir otros errores, devolvemos un error 500 y listo.
                 ServerApp.getLoggerSystem().warn("Error imprevisto: " + e.toString());
                 ServerApp.getLoggerSystem().debug(e.getMessage());
-                ServerApp.getLoggerSystem().debug(e.getStackTrace().toString());
+                ServerApp.getLoggerSystem().debug(Arrays.toString(e.getStackTrace()));
 
                 response = Response.serverError();
             }
@@ -184,14 +185,14 @@ public static Response genericAdminMethod(HttpServletRequest req, Integer paramI
                 // Detectamos errores en la SQL
                 ServerApp.getLoggerSystem().warn("Error en procesar la consulta SQL: " + e.toString());
                 ServerApp.getLoggerSystem().debug(e.getMessage());
-                ServerApp.getLoggerSystem().debug(e.getStackTrace().toString());
+                ServerApp.getLoggerSystem().debug(Arrays.toString(e.getStackTrace()));
 
                 response = Response.serverError();
             } catch (Exception e) {
                 // En caso de existir otros errores, devolvemos un error 500 y listo.
                 ServerApp.getLoggerSystem().warn("Error imprevisto: " + e.toString());
                 ServerApp.getLoggerSystem().debug(e.getMessage());
-                ServerApp.getLoggerSystem().debug(e.getStackTrace().toString());
+                ServerApp.getLoggerSystem().debug(Arrays.toString(e.getStackTrace()));
 
                 response = Response.serverError();
             }

diff --git a/src/main/resources/application.properties b/src/main/resources/application.properties
@@ -1,8 +1,15 @@
+# Path donde responderá las peticiones de la API RESTful
 spring.main.banner-mode = off
 spring.jersey.application-path = /api
 spring.jersey.type = filter
 
-mauzo.maxParallel.typeConnections = 3
+# Pagina de error personalizada
+server.error.whitelabel.enabled=false
+spring.autoconfigure.exclude=org.springframework.boot.autoconfigure.web.ErrorMvcAutoConfiguration
 
+# Configuraciones del log
 logging.level.root = INFO
-logging.pattern.console = %clr(%d{yy-MM-dd E HH:mm:ss.SSS}){blue} %clr(%-5p) %clr(${PID}){faint} %clr(---){faint} %clr([%8.15t]){cyan} %clr(%-40.40logger{0}){blue} %clr(:){red} %clr(%m){faint}%n
+logging.pattern.console = %clr(%d{yy-MM-dd E HH:mm:ss.SSS}){blue} %clr(%-5p) %clr(${PID}){faint} %clr(---){faint} %clr([%8.15t]){cyan} %clr(%-40.40logger{0}){blue} %clr(:){red} %clr(%m){faint}%n
+
+# Configuraciones propias del servidor
+mauzo.maxParallel.typeConnections = 3
diff --git a/src/main/resources/templates/error.html b/src/main/resources/templates/error.html
@@ -0,0 +1,26 @@
+<!DOCTYPE html>
+	<html>
+	  <head>
+		<meta name="viewport" content="width=device-width, initial-scale=1">
+		<meta charset="utf-8">
+		<title>Application Error</title>
+		<style media="screen">
+		  html,body,iframe {
+			margin: 0;
+			padding: 0;
+		  }
+		  html,body {
+			height: 100%;
+			overflow: hidden;
+		  }
+		  iframe {
+			width: 100%;
+			height: 100%;
+			border: 0;
+		  }
+		</style>
+	  </head>
+	  <body>
+		<iframe src="https://www.herokucdn.com/error-pages/application-error.html"></iframe>
+	  </body>
+	</html>