Validate link before opening

Signed-off-by: Julius Härtl <[email protected]> Signed-off-by: npmbuildbot-nextcloud[bot] <npmbuildbot-nextcloud[bot]@users.noreply.github.com>
nextcloud · juliusknorr · Nov 16, 2020 · Nov 12, 2020 · Nov 12, 2020 · 08dac20cec7ed0b1f01dee1ece5ee705313053a2
commit 08dac20cec7ed0b1f01dee1ece5ee705313053a2
diff --git a/js/editor-collab.js b/js/editor-collab.js
diff --git a/js/editor-collab.js.map b/js/editor-collab.js.map
diff --git a/js/editor-rich.js b/js/editor-rich.js
diff --git a/js/editor-rich.js.map b/js/editor-rich.js.map
diff --git a/js/editor.js b/js/editor.js
diff --git a/js/editor.js.map b/js/editor.js.map
diff --git a/js/files-modal.js b/js/files-modal.js
diff --git a/js/files-modal.js.map b/js/files-modal.js.map
diff --git a/js/files.js b/js/files.js
diff --git a/js/files.js.map b/js/files.js.map
diff --git a/js/public.js b/js/public.js
diff --git a/js/public.js.map b/js/public.js.map
diff --git a/js/text.js b/js/text.js
diff --git a/js/text.js.map b/js/text.js.map
diff --git a/js/vendors~editor-collab~editor-guest.js b/js/vendors~editor-collab~editor-guest.js
diff --git a/js/vendors~editor-collab~editor-guest.js.map b/js/vendors~editor-collab~editor-guest.js.map
diff --git a/js/vendors~editor-rich.js b/js/vendors~editor-rich.js
diff --git a/js/vendors~editor-rich.js.map b/js/vendors~editor-rich.js.map
diff --git a/js/vendors~editor.js b/js/vendors~editor.js
diff --git a/js/vendors~editor.js.map b/js/vendors~editor.js.map
diff --git a/js/vendors~editor~files-modal.js b/js/vendors~editor~files-modal.js
diff --git a/js/vendors~editor~files-modal.js.map b/js/vendors~editor~files-modal.js.map
diff --git a/js/viewer.js b/js/viewer.js
diff --git a/js/viewer.js.map b/js/viewer.js.map
diff --git a/src/marks/index.js b/src/marks/index.js
@@ -20,7 +20,10 @@
  *
  */
 
+import { Plugin } from 'tiptap'
+import { getMarkAttrs } from 'tiptap-utils'
 import { Bold, Italic as TipTapItalic, Strike as TipTapStrike, Link as TipTapLink } from 'tiptap-extensions'
+import { markdownit } from '../EditorFactory'
 
 /**
  * This file maps prosemirror mark names to tiptap classes,
@@ -100,6 +103,35 @@ class Link extends TipTapLink {
 		}
 	}
 
+	get plugins() {
+		if (!this.options.openOnClick) {
+			return []
+		}
+
+		return [
+			new Plugin({
+				props: {
+					handleClick: (view, pos, event) => {
+						const { schema } = view.state
+						const attrs = getMarkAttrs(view.state, schema.marks.link)
+
+						if (attrs.href && event.target instanceof HTMLAnchorElement) {
+							event.stopPropagation()
+							const htmlHref = event.target.href
+
+							if (!markdownit.validateLink(htmlHref)) {
+								console.error('Invalid link', htmlHref)
+								return
+							}
+
+							window.open(htmlHref)
+						}
+					},
+				},
+			}),
+		]
+	}
+
 }
 
 /** Strike is currently unsupported by prosemirror-markdown */