- Issue #467

Axway-API-Management-Plus · Mar 12, 2024 · ca94003 · ca94003
1 parent 77e76cf
commit ca94003
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 1 deletion.
diff --git a/modules/apim-adapter/src/main/java/com/axway/apim/lib/utils/Utils.java b/modules/apim-adapter/src/main/java/com/axway/apim/lib/utils/Utils.java
@@ -476,4 +476,8 @@ public static Map<String,String> removeEmptyValuesFromMap(Map<String, String> ma
             .filter(e -> !e.getValue().isEmpty())
             .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));
     }
+
+    public static String replaceSpecialChars(String fileName){
+        return fileName.replaceAll("[\\\\/:*?\"<>|]", "");
+    }
 }
diff --git a/modules/apim-adapter/src/test/java/com/axway/lib/utils/UtilsTest.java b/modules/apim-adapter/src/test/java/com/axway/lib/utils/UtilsTest.java
@@ -367,4 +367,10 @@ void handleAppExceptionGeneric() {
             Assert.assertEquals(99, Utils.handleAppException(exception, LoggerFactory.getLogger("Utils"), new ErrorCodeMapper()));
         }
     }
+
+    @Test
+    void replaceSplChars() {
+        Assert.assertEquals("api-old(XX) API  OLD", Utils.replaceSpecialChars("api-old/(XX) API ** OLD**"));
+        Assert.assertEquals("api-old(XX) API  OLD", Utils.replaceSpecialChars("api-old/(XX) API ** OLD**:"));
+    }
 }
diff --git a/modules/apis/src/main/java/com/axway/apim/api/export/impl/ExportHelper.java b/modules/apis/src/main/java/com/axway/apim/api/export/impl/ExportHelper.java
@@ -13,6 +13,7 @@
 import com.axway.apim.lib.EnvironmentProperties;
 import com.axway.apim.lib.error.AppException;
 import com.axway.apim.lib.error.ErrorCode;
+import com.axway.apim.lib.utils.Utils;
 import com.fasterxml.jackson.annotation.JsonInclude;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.fasterxml.jackson.databind.SerializationFeature;
@@ -59,7 +60,7 @@ public void saveAPILocally(ExportAPI exportAPI, APIResultHandler apiResultHandle
         String targetFile = null;
         String configFile;
         try {
-            targetFile = localFolder.getCanonicalPath() + "/" + exportAPI.getName() + apiDef.getAPIDefinitionType().getFileExtension();
+            targetFile = localFolder.getCanonicalPath() + "/" + Utils.replaceSpecialChars(exportAPI.getName()) + apiDef.getAPIDefinitionType().getFileExtension();
             if (!(apiDef instanceof WSDLSpecification && EnvironmentProperties.RETAIN_BACKEND_URL) && (!EnvironmentProperties.PRINT_CONFIG_CONSOLE)) {
                 writeBytesToFile(apiDef.getApiSpecificationContent(), targetFile);
                 exportAPI.getAPIDefinition().setApiSpecificationFile(exportAPI.getName() + apiDef.getAPIDefinitionType().getFileExtension());