CLOUDSTACK-333: When Datacenter name in VCenter has spaces Primary Storage 
(VMFS) discovery will fail

Description:

    Missed encoding of URI when discovering ESX cluster in vCenter.

    Signed-off-by: Vijayendra Bhamidipati <[email protected]>


Project: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/repo
Commit: 
http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/commit/8ced7da7
Tree: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/tree/8ced7da7
Diff: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/diff/8ced7da7

Branch: refs/heads/ui-plugins
Commit: 8ced7da733f8f3e9402495a239f037804ebea2a3
Parents: 2a8b9e5
Author: Vijayendra Bhamidipati <[email protected]>
Authored: Fri Oct 12 17:33:36 2012 -0400
Committer: Prasanna Santhanam <[email protected]>
Committed: Thu Dec 13 22:43:59 2012 -0800

----------------------------------------------------------------------
 .../src/com/cloud/storage/StorageManagerImpl.java  |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/blob/8ced7da7/server/src/com/cloud/storage/StorageManagerImpl.java
----------------------------------------------------------------------
diff --git a/server/src/com/cloud/storage/StorageManagerImpl.java 
b/server/src/com/cloud/storage/StorageManagerImpl.java
index e252633..a768610 100755
--- a/server/src/com/cloud/storage/StorageManagerImpl.java
+++ b/server/src/com/cloud/storage/StorageManagerImpl.java
@@ -1281,7 +1281,7 @@ public class StorageManagerImpl implements 
StorageManager, Manager, ClusterManag
         }
         URI uri = null;
         try {
-            uri = new URI(cmd.getUrl());
+            uri = new URI(UriUtils.encodeURIComponent(cmd.getUrl()));
             if (uri.getScheme() == null) {
                 throw new InvalidParameterValueException("scheme is null " + 
cmd.getUrl() + ", add nfs:// as a prefix");
             } else if (uri.getScheme().equalsIgnoreCase("nfs")) {

Reply via email to