fix: uri encoding

scalameta · Apr 5, 2022 · 7be48da · 7be48da
1 parent c3fbfe9
commit 7be48da
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 4 deletions.
diff --git a/...a/internal/metals/URIEncoderDecoder.scala → ...a/internal/metals/URIEncoderDecoder.scala b/...a/internal/metals/URIEncoderDecoder.scala → ...a/internal/metals/URIEncoderDecoder.scala
@@ -1,7 +1,8 @@
-package scala.meta.internal.metals
+package scala.meta.internal.mtags
 
 object URIEncoderDecoder {
 
+  // https://en.wikipedia.org/wiki/Percent-encoding
   private val toEscape: Map[Char, String] =
     Set('"', '<', '>', '&', '\'', '[', ']', '{', '}', ' ')
       .map(char => char -> ("%" + char.toHexString))

diff --git a/mtags/src/main/scala-2/scala/meta/internal/mtags/MtagsEnrichments.scala b/mtags/src/main/scala-2/scala/meta/internal/mtags/MtagsEnrichments.scala
@@ -133,8 +133,13 @@ trait MtagsEnrichments extends CommonMtagsEnrichments {
         URLDecoder.decode(value, "UTF-8").toAbsolutePath(followSymlink)
       else if (value.toUpperCase.startsWith("JAR:FILE%3A"))
         URLDecoder.decode(value, "UTF-8").toAbsolutePath(followSymlink)
-      else
-        URI.create(value.stripPrefix("metals:")).toAbsolutePath(followSymlink)
+      else if (value.toUpperCase.startsWith("JAR"))
+        URI.create(value).toAbsolutePath(followSymlink)
+      else {
+        val stripped = value.stripPrefix("metals:")
+        val percentEncoded = URIEncoderDecoder.encode(stripped)
+        URI.create(percentEncoded).toAbsolutePath(followSymlink)
+      }
     }
     def lastIndexBetween(
         char: Char,

diff --git a/tests/unit/src/test/scala/tests/MetalsEnrichmentsSuite.scala b/tests/unit/src/test/scala/tests/MetalsEnrichmentsSuite.scala
@@ -22,10 +22,17 @@ class MetalsEnrichmentsSuite extends BaseSuite {
     assertEquals(relativeCreated, expected)
   }
 
-  test("uri-to-absolute-path") {
+  test("dont-decode-uri") {
     val uri =
       "file:///Users/happyMetalsUser/hello%20space%20world/src/main/scala/Main.scala"
     val path = uri.toAbsolutePath
     assert(path.toString().contains("hello space world"))
   }
+
+  test("encode-uri-space") {
+    val uri =
+      "file:///Users/happyMetalsUser/hello space+world/src/main/scala/Main.scala"
+    val path = uri.toAbsolutePath
+    assert(path.toString().contains("hello space+world"))
+  }
 }