From 2629949b669c6e14397d6a6d4dd35ef9df9451c4 Mon Sep 17 00:00:00 2001
From: David <david.gautier-ext@pole-emploi.fr>
Date: Wed, 18 Oct 2023 15:15:22 +0200
Subject: [PATCH] fix : URI_DETECTOR regex

---
 .../convert/confluence/ReferenceProvider.kt   | 16 ++++++++++---
 .../confluence/ReferenceProviderImplTest.kt   | 23 +++++++++++++++++++
 2 files changed, 36 insertions(+), 3 deletions(-)

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 1bb5e314..80e92b87 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
@@ -2,8 +2,11 @@ package com.github.zeldigas.text2confl.convert.confluence
 
 import com.github.zeldigas.text2confl.convert.PageHeader
 import io.github.oshai.kotlinlogging.KotlinLogging
+import java.net.URL
 import java.nio.file.InvalidPathException
 import java.nio.file.Path
+import java.util.regex.Matcher
+import java.util.regex.Pattern
 import kotlin.io.path.relativeTo
 
 interface ReferenceProvider {
@@ -41,16 +44,23 @@ class ReferenceProviderImpl(private val basePath: Path, documents: Map<Path, Pag
     ReferenceProvider {
 
     companion object {
-        private val URI_DETECTOR = "^(https?|ftp)://[-a-zA-Z0-9+&@#/%?=~_|!:,.;]*[-a-zA-Z0-9+&@#/%=~_|]".toRegex()
+        private const val URI_DETECTOR = "^(https?|ftp)://[-a-zA-Z0-9+&@#/%?=~_|!:,.;]*[-a-zA-Z0-9+&@#/%=~_|]"
+        private const val MAILTO_DETECTOR = "mailto:"
         private val logger = KotlinLogging.logger {}
     }
+    fun isValid(url: String): Boolean {
+        val pattern = Pattern.compile(URI_DETECTOR, Pattern.CASE_INSENSITIVE);
+        val matcher = pattern.matcher(url.trim());
+        return matcher.matches();
+    }
 
     private val normalizedDocs =
         documents.map { (path, header) -> path.relativeTo(basePath).normalize() to header }.toMap()
 
     override fun resolveReference(source: Path, refTo: String): Reference? {
 
-        if (URI_DETECTOR.matches(refTo)) return null
+        if (refTo.startsWith(MAILTO_DETECTOR)) return null
+        if (isValid(refTo)) return null
         if (refTo.startsWith("#")) return Anchor(refTo.substring(1))
 
         val parts = refTo.split("#", limit = 2)
@@ -62,7 +72,7 @@ class ReferenceProviderImpl(private val basePath: Path, documents: Map<Path, Pag
             val targetPath = source.resolveSibling(ref).relativeTo(basePath).normalize()
             val document = normalizedDocs[targetPath]?.title ?: return null
             return Xref(document, anchor)
-        } catch (ex: InvalidPathException){
+        } catch (ex: InvalidPathException) {
             logger.error { "Failed to resolve : $refTo  from $source" }
             throw ex
         }
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 8e1192bf..46ed16e5 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
@@ -3,6 +3,7 @@ package com.github.zeldigas.text2confl.convert.confluence
 import assertk.assertThat
 import assertk.assertions.isEqualTo
 import assertk.assertions.isNotNull
+import assertk.assertions.isNull
 import com.github.zeldigas.text2confl.convert.PageHeader
 import org.junit.jupiter.api.Test
 import org.junit.jupiter.params.ParameterizedTest
@@ -68,4 +69,26 @@ internal class ReferenceProviderImplTest {
 
         assertThat(result).isNotNull().isEqualTo(Xref("Sub Title One", "test"))
     }
+
+
+    @Test
+    internal fun `Http resolution`() {
+        val result = providerImpl.resolveReference(Path("docs/one.md"), "http://github.com")
+
+        assertThat(result).isNull()
+    }
+
+    @Test
+    internal fun `Https resolution`() {
+        val result = providerImpl.resolveReference(Path("docs/one.md"), "https://github.com")
+
+        assertThat(result).isNull()
+    }
+
+    @Test
+    internal fun `Mailto resolution`() {
+        val result = providerImpl.resolveReference(Path("docs/one.md"), "mailto:john.doe@github.com")
+
+        assertThat(result).isNull()
+    }
 }
\ No newline at end of file