ignore hrefs with a scheme - fixes #2909

sveltejs · Nov 25, 2021 · b4d420f · b4d420f
1 parent c6b9ee4
commit b4d420f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/packages/kit/src/utils/url.js b/packages/kit/src/utils/url.js
@@ -1,10 +1,13 @@
 const absolute = /^([a-z]+:)?\/?\//;
+const scheme = /^[a-z]+:/;
 
 /**
  * @param {string} base
  * @param {string} path
  */
 export function resolve(base, path) {
+	if (scheme.test(path)) return path;
+
 	const base_match = absolute.exec(base);
 	const path_match = absolute.exec(path);
 

diff --git a/packages/kit/src/utils/url.spec.js b/packages/kit/src/utils/url.spec.js
@@ -46,4 +46,8 @@ test('resolves an absolute path', () => {
 	assert.equal(resolve('/a/b/c', 'https://example.com/foo'), 'https://example.com/foo');
 });
 
+test('handles schemes like tel: and mailto:', () => {
+	assert.equal(resolve('/a/b/c', 'mailto:hello@svelte.dev'), 'mailto:hello@svelte.dev');
+});
+
 test.run();