Fix separator character

etr · etr · commit 6aaa01823782 · 2019-01-23T04:46:47.000Z
diff --git a/src/http_utils.cpp b/src/http_utils.cpp
@@ -225,7 +225,7 @@ std::string http_utils::standardize_url(const std::string& url)
 {
     std::string n_url = url;
 
-    std::string::iterator new_end = std::unique(n_url.begin(), n_url.end(), [](char a, char b) { return (a == b) && (a == ' '); });
+    std::string::iterator new_end = std::unique(n_url.begin(), n_url.end(), [](char a, char b) { return (a == b) && (a == '/'); });
     n_url.erase(new_end, n_url.end());
 
     std::string::size_type n_url_length = n_url.length();
diff --git a/test/unit/http_utils_test.cpp b/test/unit/http_utils_test.cpp
@@ -128,6 +128,10 @@ LT_BEGIN_AUTO_TEST(http_utils_suite, standardize_url)
     url = "/abc/pqr", result = "";
     result = http::http_utils::standardize_url(url);
     LT_CHECK_EQ(result, "/abc/pqr");
+
+    url = "/abc//pqr", result = "";
+    result = http::http_utils::standardize_url(url);
+    LT_CHECK_EQ(result, "/abc/pqr");
 LT_END_AUTO_TEST(standardize_url)
 
 LT_BEGIN_AUTO_TEST(http_utils_suite, ip_to_str)

Original file line number	Diff line number	Diff line change
`@@ -225,7 +225,7 @@ std::string http_utils::standardize_url(const std::string& url)`
`225`	`225`	`{`
`226`	`226`	`std::string n_url = url;`
`227`	`227`
`228`		`- std::string::iterator new_end = std::unique(n_url.begin(), n_url.end(), [](char a, char b) { return (a == b) && (a == ' '); });`
	`228`	`+ std::string::iterator new_end = std::unique(n_url.begin(), n_url.end(), [](char a, char b) { return (a == b) && (a == '/'); });`
`229`	`229`	`n_url.erase(new_end, n_url.end());`
`230`	`230`
`231`	`231`	`std::string::size_type n_url_length = n_url.length();`