Fix a bug in the removeDots implementation.

Motivation:

HttpUtils#removeDots does not properly handle the C. rule of section 5.2.4 of RFC3986. When the output buffer
does not contain a / it should discard the entire content of the buffer.

Changes:

When handling rule C in HttpUtils#removeDots, discard the output buffer when no / is present.

Author: vietj

src/main/java/io/vertx/core/http/impl/HttpUtils.java

@@ -406,16 +406,12 @@ public static String removeDots(CharSequence path) {
         // preserve last slash
         i += 3;
         int pos = obuf.lastIndexOf("/");
-        if (pos != -1) {
-          obuf.delete(pos, obuf.length());
-        }
+        obuf.setLength(pos == -1 ? 0 : pos);
       } else if (matches(path, i, "/..", true)) {
         path = "/";
         i = 0;
         int pos = obuf.lastIndexOf("/");
-        if (pos != -1) {
-          obuf.delete(pos, obuf.length());
-        }
+        obuf.setLength(pos == -1 ? 0 : pos);
       } else if (matches(path, i, ".", true) || matches(path, i, "..", true)) {
         break;
       } else {

src/test/java/io/vertx/core/impl/HttpUtilsTest.java

@@ -0,0 +1,68 @@
+/*
+ * Copyright (c) 2011-2023 Contributors to the Eclipse Foundation
+ *
+ * This program and the accompanying materials are made available under the
+ * terms of the Eclipse Public License 2.0 which is available at
+ * http://www.eclipse.org/legal/epl-2.0, or the Apache License, Version 2.0
+ * which is available at https://www.apache.org/licenses/LICENSE-2.0.
+ *
+ * SPDX-License-Identifier: EPL-2.0 OR Apache-2.0
+ */
+package io.vertx.core.impl;
+
+import io.vertx.core.http.impl.HttpUtils;
+import org.junit.Test;
+
+import static org.junit.Assert.assertEquals;
+
+public class HttpUtilsTest {
+
+  @Test
+  public void testRemoveDotSegmentsRuleA() {
+    assertDotSegments("", "../");
+    assertDotSegments("", "./");
+
+    assertDotSegments("foo", "../foo");
+    assertDotSegments("foo", "./foo");
+  }
+
+  @Test
+  public void testRemoveDotSegmentsRuleB() {
+    assertDotSegments("/", "/./");
+    assertDotSegments("/", "/.");
+
+    assertDotSegments("/foo", "/./foo");
+  }
+
+  @Test
+  public void testRemoveDotSegmentsRuleC() {
+    assertDotSegments("/", "/../");
+    assertDotSegments("/foo", "/../foo");
+    assertDotSegments("/", "/..");
+    assertDotSegments("/", "/foo/../");
+    assertDotSegments("/", "/foo/..");
+    assertDotSegments("/", "foo/../");
+    assertDotSegments("/", "foo/..");
+    assertDotSegments("/foo/", "/foo/bar/../");
+    assertDotSegments("/foo/", "/foo/bar/..");
+    assertDotSegments("foo/", "foo/bar/../");
+    assertDotSegments("foo/", "foo/bar/..");
+  }
+
+  @Test
+  public void testRemoveDotSegmentsRuleD() {
+    assertDotSegments("", ".");
+    assertDotSegments("", "..");
+  }
+
+  @Test
+  public void testRemoveDotSegmentsRuleE() {
+    assertDotSegments("/foo", "/foo");
+    assertDotSegments("foo", "foo");
+  }
+
+  private static void assertDotSegments(String expected, String test) {
+    String actual = HttpUtils.removeDots(test);
+    assertEquals(expected, actual);
+  }
+}