Fix issue adding external links

Author: gflores-jahia

src/main/java/org/jahia/modules/richtext/configuration/parse/Parser.java

@@ -19,11 +19,14 @@ private enum PolicyType {
 
     static {
         PATTERNS = new HashMap<>();
+        String onsiteUrl = "(?:[\\p{L}\\p{N}\\\\\\.\\#@\\$%\\+&;\\-_~,\\?=/!{}:]+|\\#(\\w)+)";
+        String offsiteUrl = "(\\s*(?:(?:ht|f)tps?://|mailto:)[\\p{L}\\p{N}][\\p{L}\\p{N}\\p{Zs}\\.\\#@\\$%\\+&;:\\-_~,\\?=/!\\(\\)"
+                + "]*+\\s*)";
         PATTERNS.put("NUMBER_OR_PERCENT", Pattern.compile("[0-9]+%?"));
-        PATTERNS.put("ONSITE_URL", Pattern.compile("(?:[\\p{L}\\p{N}\\\\\\.\\#@\\$%\\+&;\\-_~,\\?=/!{}:]+|\\#(\\w)+)"));
+        PATTERNS.put("ONSITE_URL", Pattern.compile(onsiteUrl));
+        PATTERNS.put("OFFSITE_URL", Pattern.compile(onsiteUrl));
+        PATTERNS.put("LINKS_URL", Pattern.compile(String.format("(?:%s|%s)", onsiteUrl, offsiteUrl)));
         PATTERNS.put("HTML_ID", Pattern.compile("[a-zA-Z0-9\\:\\-_\\.]+"));
-        PATTERNS.put("OFFSITE_URL", Pattern.compile("\\s*(?:(?:ht|f)tps?://|mailto:)[\\p{L}\\p{N}]"
-                        + "[\\p{L}\\p{N}\\p{Zs}\\.\\#@\\$%\\+&;:\\-_~,\\?=/!\\(\\)]*+\\s*"));
         PATTERNS.put("HTML_CLASS", Pattern.compile("[a-zA-Z0-9\\s,\\-_]+"));
         PATTERNS.put("NUMBER", Pattern.compile("[+-]?(?:(?:[0-9]+(?:\\.[0-9]*)?)|\\.[0-9]+)"));
         PATTERNS.put("NAME", Pattern.compile("[a-zA-Z0-9\\-_\\$]+"));

src/main/resources/META-INF/configuration-default/org.jahia.modules.richtext.config-default.yml

@@ -38,7 +38,7 @@ htmlFiltering:
       pattern: NUMBER_OR_PERCENT
       elements: canvas, img, table, td, th, col, colgroup, video
     - name: href
-      pattern: ONSITE_URL
+      pattern: LINKS_URL
       elements: a
     - name: hreflang
       elements: a

tests/cypress/e2e/defaultFiltering.cy.ts

@@ -29,13 +29,29 @@ describe('Default HTML filtering', () => {
         deleteNode(path);
     });
 
-    it('allows asset links', () => {
+    it('allows internal links', () => {
         // Note that the actual href text being sent over to the sanitizer is '##doc-context##/{workspace}/##ref:link1##'
         const text = '<p><a href="/files/{workspace}/sites/digitall/files/images/pdf/Conference%20Guide.pdf" ' +
             'title="Conference Guide.pdf">/files/{workspace}/sites/digitall/files/images/pdf/Conference%20Guide.pdf</a></p>';
         modifyContent(path, text);
         getContent(path).then(result => {
-            expect(result.data.jcr.nodeByPath.property.value).to.contain('href');
+            const value = result.data.jcr.nodeByPath.property.value;
+            expect(value).to.contain('<p>');
+            expect(value).to.contain('<a');
+            expect(value).to.contain('href');
+            expect(value).to.contain('title');
+        });
+    });
+
+    it('allows external links', () => {
+        const text = '<p>This is a <a href="http://google.com" title="My google link">google link</a></p>';
+        modifyContent(path, text);
+        getContent(path).then(result => {
+            const value = result.data.jcr.nodeByPath.property.value;
+            expect(value).to.contain('<p>');
+            expect(value).to.contain('<a');
+            expect(value).to.contain('href');
+            expect(value).to.contain('title');
         });
     });
 });