Merge pull request #1808, fix URL parsing problem when user filed con…

…tains '@' characters.

Fixed #1470

dubbo-common/src/main/java/com/alibaba/dubbo/common/URL.java

@@ -224,7 +224,7 @@ public static URL valueOf(String url) {
             path = url.substring(i + 1);
             url = url.substring(0, i);
         }
-        i = url.indexOf("@");
+        i = url.lastIndexOf("@");
         if (i >= 0) {
             username = url.substring(0, i);
             int j = username.indexOf(":");

dubbo-common/src/test/java/com/alibaba/dubbo/common/URLTest.java

@@ -637,4 +637,19 @@ public void testAddParameters() throws Exception {
         parameters.put("version", null);
         url.addParameters(parameters);
     }
+
+    @Test
+    public void testUserNamePasswordContainsAt(){
+        // Test username or password contains "@"
+        URL url = URL.valueOf("ad@min:hello@[email protected]:20880/context/path?version=1.0.0&application=morgan");
+        assertNull(url.getProtocol());
+        assertEquals("ad@min", url.getUsername());
+        assertEquals("hello@1234", url.getPassword());
+        assertEquals("10.20.130.230", url.getHost());
+        assertEquals(20880, url.getPort());
+        assertEquals("context/path", url.getPath());
+        assertEquals(2, url.getParameters().size());
+        assertEquals("1.0.0", url.getParameter("version"));
+        assertEquals("morgan", url.getParameter("application"));
+    }
 }