Allow dashes according to RFC in all domain parts except TLD. Fixes #63

Author: Jikstra

lib/re.js

@@ -101,10 +101,7 @@ module.exports = function (opts) {
       '|' +
       '(?:' + re.src_pseudo_letter + ')' +
       '|' +
-      // don't allow `--` in domain names, because:
-      // - that can conflict with markdown — / –
-      // - nobody use those anyway
-      '(?:' + re.src_pseudo_letter + '(?:-(?!-)|' + re.src_pseudo_letter + '){0,61}' + re.src_pseudo_letter + ')' +
+      '(?:' + re.src_pseudo_letter + '(?:-|' + re.src_pseudo_letter + '){0,61}' + re.src_pseudo_letter + ')' +
     ')';
 
   re.src_host =

test/fixtures/links.txt

@@ -302,3 +302,15 @@ test@google.com
 
 ｜http://google.com｜bar
 http://google.com
+
+%
+% Domains with multiple dashes
+%
+
+https://5b0ee223b312746c1659db3f--thelounge-chat.netlify.com/docs/
+
+www.a--b.com
+
+www.c--u.com
+
+http://a---b.com/

test/fixtures/not_links.txt

@@ -9,7 +9,6 @@ hppt://example.com
 example.coma
 -example.coma
 foo.123
-http://a.b--c.de/   % `--` disabled, because collision possible
 localhost           % only with protocol allowed
 localhost/
 ///localhost        % 3 '/' not allowed
@@ -50,4 +49,4 @@ a1.2.3.4
 % Not email
 %
 foo@bar     % Should be at second level domain & with correct tld
-mailto:bar
+mailto:bar