Merge pull request #260 from MihaZupan/fix-autolinks

Allow #, ? and : instead of only / as the domain separator for AutoLinks
xoofx · Oct 25, 2018 · 92b06f0 · 92b06f0
2 parents 07e6a13 + ecd625d
commit 92b06f0
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 2 deletions.
diff --git a/src/Markdig.Tests/Specs/AutoLinks.md b/src/Markdig.Tests/Specs/AutoLinks.md
@@ -1,4 +1,4 @@
-# Extensions
+# Extensions
 
 This section describes the different extensions supported:
 
@@ -219,4 +219,24 @@ Domain names that contain invalid characters (not AlphaNumberic, -, _ or .) won'
 https://[your-domain]/api
 .
 <p>https://[your-domain]/api</p>
+````````````````````````````````
+
+Domain names followed by ?, : or # instead of / are matched
+
+```````````````````````````````` example
+https:?
+
+https:?a
+
+https:#a
+
+https::
+
+https::443
+.
+<p><a href="https:">https:</a>?</p>
+<p><a href="https:?a">https:?a</a></p>
+<p><a href="https:#a">https:#a</a></p>
+<p><a href="https:">https:</a>:</p>
+<p><a href="https::443">https::443</a></p>
 ````````````````````````````````
diff --git a/src/Markdig/Helpers/LinkHelper.cs b/src/Markdig/Helpers/LinkHelper.cs
@@ -690,7 +690,7 @@ public static bool IsValidDomain(string link, int prefixLength)
 
  if (!c.IsAlphaNumeric())
  {
- if (c == '/') // End of domain name
+ if (c == '/' || c == '?' || c == '#' || c == ':') // End of domain name
  break;
 
  if (c == '_')