- avoid printing exception for help command

Axway-API-Management-Plus · Mar 25, 2024 · dab46ff · dab46ff
1 parent 98989c1
commit dab46ff
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/modules/apim-adapter/src/main/java/com/axway/apim/lib/CLIOptions.java b/modules/apim-adapter/src/main/java/com/axway/apim/lib/CLIOptions.java
@@ -89,7 +89,7 @@ public void parse() throws AppException {
             this.envProperties = new EnvironmentProperties(cmd.getOptionValue("stage"), getValue("apimCLIHome"));
         } catch (ParseException e) {
             printUsage(e.getMessage(), args);
-            throw new AppException(e.getMessage(), ErrorCode.UNXPECTED_ERROR, e);
+            throw new AppException(e.getMessage(), ErrorCode.CLI_PARSING, e);
         }
     }
 

diff --git a/modules/apim-adapter/src/main/java/com/axway/apim/lib/error/AppException.java b/modules/apim-adapter/src/main/java/com/axway/apim/lib/error/AppException.java
@@ -47,7 +47,7 @@ public ErrorCode getError() {
     }
 
     public void logException(Logger logger) {
-        if (error == ErrorCode.SUCCESS)
+        if (error == ErrorCode.SUCCESS || error == ErrorCode.CLI_PARSING)
             return;
         Throwable cause = null;
         if (error.getPrintStackTrace().booleanValue() || logger.isDebugEnabled()) {

diff --git a/modules/apim-adapter/src/main/java/com/axway/apim/lib/error/ErrorCode.java b/modules/apim-adapter/src/main/java/com/axway/apim/lib/error/ErrorCode.java
@@ -6,6 +6,8 @@ public enum ErrorCode {
 
     SUCCESS(0, "Replication of the API was successful.", false, LogLevel.INFO),
     UNSUPPORTED_API_SPECIFICATION(1, "The API-Specification is not supported by the APIM-CLI.", false),
+    CLI_PARSING(2, "Error in parsing input parameters", false, LogLevel.INFO),
+
     MISSING_PARAMETER(5, "There is a missing parameter.", false),
     INVALID_PARAMETER(6, "There is an invalid parameter.", false),
     API_ALREADY_EXISTS(7, "The API already exists for another organization.", false),