From c7db93cc3cc73eb3b8dc858c81b5aae8070c21b2 Mon Sep 17 00:00:00 2001
From: David <david.gautier-ext@pole-emploi.fr>
Date: Wed, 18 Oct 2023 20:26:52 +0200
Subject: [PATCH] fix : URI_DETECTOR regex, added custom detector localhost and
 mailto

---
 .../convert/confluence/ReferenceProvider.kt   |  5 +++-
 .../confluence/ReferenceProviderImplTest.kt   | 23 +++++++++++++++++++
 2 files changed, 27 insertions(+), 1 deletion(-)

diff --git a/convert/src/main/kotlin/com/github/zeldigas/text2confl/convert/confluence/ReferenceProvider.kt b/convert/src/main/kotlin/com/github/zeldigas/text2confl/convert/confluence/ReferenceProvider.kt
index 80e92b87..9b34908e 100644
--- a/convert/src/main/kotlin/com/github/zeldigas/text2confl/convert/confluence/ReferenceProvider.kt
+++ b/convert/src/main/kotlin/com/github/zeldigas/text2confl/convert/confluence/ReferenceProvider.kt
@@ -44,8 +44,10 @@ class ReferenceProviderImpl(private val basePath: Path, documents: Map<Path, Pag
     ReferenceProvider {
 
     companion object {
-        private const val URI_DETECTOR = "^(https?|ftp)://[-a-zA-Z0-9+&@#/%?=~_|!:,.;]*[-a-zA-Z0-9+&@#/%=~_|]"
+        private const val URI_DETECTOR = "^(https?|ftp)://[-a-zA-Z0-9+&@#/%?=~_|!:,.;'()]*[-a-zA-Z0-9+&@#/%=~_|]"
         private const val MAILTO_DETECTOR = "mailto:"
+        private const val LOCALHOST_DETECTOR = "localhost:"
+
         private val logger = KotlinLogging.logger {}
     }
     fun isValid(url: String): Boolean {
@@ -60,6 +62,7 @@ class ReferenceProviderImpl(private val basePath: Path, documents: Map<Path, Pag
     override fun resolveReference(source: Path, refTo: String): Reference? {
 
         if (refTo.startsWith(MAILTO_DETECTOR)) return null
+        if (refTo.startsWith(LOCALHOST_DETECTOR)) return null
         if (isValid(refTo)) return null
         if (refTo.startsWith("#")) return Anchor(refTo.substring(1))
 
diff --git a/convert/src/test/kotlin/com/github/zeldigas/text2confl/convert/confluence/ReferenceProviderImplTest.kt b/convert/src/test/kotlin/com/github/zeldigas/text2confl/convert/confluence/ReferenceProviderImplTest.kt
index 46ed16e5..21b90b6b 100644
--- a/convert/src/test/kotlin/com/github/zeldigas/text2confl/convert/confluence/ReferenceProviderImplTest.kt
+++ b/convert/src/test/kotlin/com/github/zeldigas/text2confl/convert/confluence/ReferenceProviderImplTest.kt
@@ -91,4 +91,27 @@ internal class ReferenceProviderImplTest {
 
         assertThat(result).isNull()
     }
+
+
+    @Test
+    internal fun `French url resolution`() {
+        val result = providerImpl.resolveReference(Path("docs/one.md"), "http://github.com/handle'case")
+
+        assertThat(result).isNull()
+    }
+
+    @Test
+    internal fun `Parenthesis url resolution`() {
+        val result = providerImpl.resolveReference(Path("docs/one.md"), "http://github.com/handle()case")
+
+        assertThat(result).isNull()
+    }
+
+    @Test
+    internal fun `localhost url resolution`() {
+        val result = providerImpl.resolveReference(Path("docs/one.md"), "localhost:9000")
+
+        assertThat(result).isNull()
+    }
+
 }
\ No newline at end of file