[geode] branch develop updated: GEODE-5463: Remove invalid path chars for Windows

[jensdeppe]

This is an automated email from the ASF dual-hosted git repository.

jensdeppe pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/geode.git


The following commit(s) were added to refs/heads/develop by this push:
     new 6cedd25  GEODE-5463: Remove invalid path chars for Windows
6cedd25 is described below

commit 6cedd257e2cf648f638289ecf30726737da793d7
Author: Jens Deppe <jde...@pivotal.io>
AuthorDate: Mon Jul 23 13:14:43 2018 -0700

    GEODE-5463: Remove invalid path chars for Windows
---
 .../org/apache/geode/internal/cache/backup/BackupWriterFactory.java     | 2 ++
 1 file changed, 2 insertions(+)

diff --git 
a/geode-core/src/main/java/org/apache/geode/internal/cache/backup/BackupWriterFactory.java
 
b/geode-core/src/main/java/org/apache/geode/internal/cache/backup/BackupWriterFactory.java
index 085c1fb..c19605d 100644
--- 
a/geode-core/src/main/java/org/apache/geode/internal/cache/backup/BackupWriterFactory.java
+++ 
b/geode-core/src/main/java/org/apache/geode/internal/cache/backup/BackupWriterFactory.java
@@ -26,6 +26,8 @@ enum BackupWriterFactory {
   FILE_SYSTEM("FileSystem") {
     @Override
     BackupWriter createWriter(Properties properties, String memberId) {
+      // Remove chars that are illegal in Windows paths
+      memberId = memberId.replaceAll("[:()]", "-");
       FileSystemBackupWriterConfig config = new 
FileSystemBackupWriterConfig(properties);
       Path targetDir = Paths.get(config.getTargetDirectory())
           .resolve(properties.getProperty(TIMESTAMP)).resolve(memberId);