Fix path for a smb global mapping to allow mount the share more that once

Author: vitaliy-leschenko

.gitignore

@@ -1,3 +1,4 @@
 /bin/
 settings.json
 integrationtests/integrationtests.test.exe
+.vs/

pkg/server/smb/server.go

@@ -25,6 +25,20 @@ func normalizeWindowsPath(path string) string {
 	return normalizedPath
 }
 
+func normalizeMappingPath(path string) string {
+	items := strings.Split(path, "\\")
+	parts := []string{}
+	for _, s := range items {
+		if len(s) > 0 {
+			parts = append(parts, s)
+			if len(parts) == 2 {
+				break
+			}
+		}
+	}
+	return strings.ToLower("\\\\" + parts[0] + "\\" + parts[1])
+}
+
 func NewServer(hostAPI smb.API, fsServer *fsserver.Server) (*Server, error) {
 	return &Server{
 		hostAPI:  hostAPI,
@@ -43,38 +57,45 @@ func (s *Server) NewSmbGlobalMapping(context context.Context, request *internal.
 		return response, fmt.Errorf("remote path is empty")
 	}
 
-	isMapped, err := s.hostAPI.IsSmbMapped(remotePath)
+	mappingPath := normalizeMappingPath(remotePath)
+
+	isMapped, err := s.hostAPI.IsSmbMapped(mappingPath)
 	if err != nil {
 		isMapped = false
 	}
 
 	if isMapped {
-		valid, err := s.fsServer.PathValid(context, remotePath)
+		klog.V(4).Infof("Remote %s already mapped. Validating...", mappingPath)
+
+		valid, err := s.fsServer.PathValid(context, mappingPath)
 		if err != nil {
-			klog.Warningf("PathValid(%s) failed with %v, ignore error", remotePath, err)
+			klog.Warningf("PathValid(%s) failed with %v, ignore error", mappingPath, err)
 		}
 
 		if !valid {
-			klog.V(4).Infof("RemotePath %s is not valid, removing now", remotePath)
-			err := s.hostAPI.RemoveSmbGlobalMapping(remotePath)
+			klog.V(4).Infof("RemotePath %s is not valid, removing now", mappingPath)
+			err := s.hostAPI.RemoveSmbGlobalMapping(mappingPath)
 			if err != nil {
-				klog.Errorf("RemoveSmbGlobalMapping(%s) failed with %v", remotePath, err)
+				klog.Errorf("RemoveSmbGlobalMapping(%s) failed with %v", mappingPath, err)
 				return response, err
 			}
 			isMapped = false
+		} else {
+			klog.V(4).Infof("RemotePath %s is valid", mappingPath)
 		}
 	}
 
 	if !isMapped {
-		klog.V(4).Infof("Remote %s not mapped. Mapping now!", remotePath)
-		err := s.hostAPI.NewSmbGlobalMapping(remotePath, request.Username, request.Password)
+		klog.V(4).Infof("Remote %s not mapped. Mapping now!", mappingPath)
+		err := s.hostAPI.NewSmbGlobalMapping(mappingPath, request.Username, request.Password)
 		if err != nil {
 			klog.Errorf("failed NewSmbGlobalMapping %v", err)
 			return response, err
 		}
 	}
 
 	if len(localPath) != 0 {
+		klog.V(4).Infof("ValidatePluginPath: '%s'", localPath)
 		err = s.fsServer.ValidatePluginPath(localPath)
 		if err != nil {
 			klog.Errorf("failed validate plugin path %v", err)
@@ -101,11 +122,13 @@ func (s *Server) RemoveSmbGlobalMapping(context context.Context, request *intern
 		return response, fmt.Errorf("remote path is empty")
 	}
 
-	err := s.hostAPI.RemoveSmbGlobalMapping(remotePath)
+	mappingPath := normalizeMappingPath(remotePath)
+	err := s.hostAPI.RemoveSmbGlobalMapping(mappingPath)
 	if err != nil {
 		klog.Errorf("failed RemoveSmbGlobalMapping %v", err)
 		return response, err
 	}
+
 	klog.V(2).Infof("RemoveSmbGlobalMapping on remote path %q is completed", request.RemotePath)
 	return response, nil
 }