fix: urlPaser will incorrect parsing url like http://abc.com/xxx@xxx/a/b

 (#1511)

Co-authored-by: Lalit Kumar Bhasin <[email protected]>

ext/include/opentelemetry/ext/http/common/url_parser.h

@@ -52,11 +52,16 @@ class UrlParser
     }
 
     // credentials
-    pos = url_.find_first_of("@", cpos);
-    if (pos != std::string::npos)
+    size_t pos1 = url_.find_first_of("@", cpos);
+    size_t pos2 = url_.find_first_of("/", cpos);
+    if (pos1 != std::string::npos)
     {
       // TODO - handle credentials
-      cpos = pos + 1;
+      if (pos2 == std::string::npos || pos1 < pos2)
+      {
+        pos  = pos1;
+        cpos = pos1 + 1;
+      }
     }
     pos          = url_.find_first_of(":", cpos);
     bool is_port = false;
@@ -129,4 +134,4 @@ class UrlParser
 
 }  // namespace http
 }  // namespace ext
-OPENTELEMETRY_END_NAMESPACE
+OPENTELEMETRY_END_NAMESPACE

ext/test/http/url_parser_test.cc

@@ -113,6 +113,13 @@ TEST(UrlParserTests, BasicTests)
         {"path", "/path1/path2"},
         {"query", "q1=a1&q2=a2"},
         {"success", "true"}}},
+      {"http://www.abc.com/path1@bbb/path2?q1=a1&q2=a2",
+       {{"host", "www.abc.com"},
+        {"port", "80"},
+        {"scheme", "http"},
+        {"path", "/path1@bbb/path2"},
+        {"query", "q1=a1&q2=a2"},
+        {"success", "true"}}},
 
   };
   for (auto &url_map : urls_map)