Sonar fixes for API to retrieve UI configuration 80/98980/2
authorilanap <ilanap@amdocs.com>
Sun, 1 Dec 2019 08:10:33 +0000 (10:10 +0200)
committerOren Kleks <oren.kleks@amdocs.com>
Sun, 1 Dec 2019 09:44:59 +0000 (09:44 +0000)
Added configuration file with the menu configuration from catalog-ui. Added to the configuration manager. to chef and to the ui rest APIs.

Issue-ID: SDC-2663
Signed-off-by: ilanap <ilanap@amdocs.com>
Change-Id: I032a7666f470e7d4797a555470bf9eb4a43cfcc4

catalog-fe/src/main/java/org/openecomp/sdc/fe/servlets/ConfigServlet.java

index 08b7c43..f3880ed 100644 (file)
@@ -48,6 +48,8 @@ import javax.ws.rs.core.Response.Status;
 public class ConfigServlet extends LoggingServlet {
 
     private static final Logger LOGGER = LoggerFactory.getLogger(ConfigServlet.class.getName());
+    public static final String UNEXPECTED_FE_RESPONSE_LOGGING_ERROR = "Unexpected FE response logging error :";
+    public static final String ERROR_FE_RESPONSE = "FE Response";
 
     @GET
     @Path("/ui/workspace")
@@ -73,8 +75,8 @@ public class ConfigServlet extends LoggingServlet {
 
             return response;
         } catch (Exception e) {
-            FeEcompErrorManager.getInstance().logFeHttpLoggingError("FE Response");
-            LOGGER.error("Unexpected FE response logging error :", e);
+            FeEcompErrorManager.getInstance().logFeHttpLoggingError(ERROR_FE_RESPONSE);
+            LOGGER.error(UNEXPECTED_FE_RESPONSE_LOGGING_ERROR, e);
             return Response.status(Status.INTERNAL_SERVER_ERROR).entity("{}").build();
         }
 
@@ -101,8 +103,8 @@ public class ConfigServlet extends LoggingServlet {
 
             return response;
         } catch (Exception e) {
-            FeEcompErrorManager.getInstance().logFeHttpLoggingError("FE Response");
-            LOGGER.error("Unexpected FE response logging error :", e);
+            FeEcompErrorManager.getInstance().logFeHttpLoggingError( ERROR_FE_RESPONSE);
+            LOGGER.error(UNEXPECTED_FE_RESPONSE_LOGGING_ERROR, e);
             return Response.status(Status.INTERNAL_SERVER_ERROR).entity("{}").build();
         }
 
@@ -133,8 +135,8 @@ public class ConfigServlet extends LoggingServlet {
 
             return response;
         } catch (Exception e) {
-            FeEcompErrorManager.getInstance().logFeHttpLoggingError("FE Response");
-            LOGGER.error("Unexpected FE response logging error :", e);
+            FeEcompErrorManager.getInstance().logFeHttpLoggingError(ERROR_FE_RESPONSE);
+            LOGGER.error(UNEXPECTED_FE_RESPONSE_LOGGING_ERROR, e);
             return Response.status(Status.INTERNAL_SERVER_ERROR).entity("{}").build();
         }
     }