Fix path segments containing colons being misidentified as absolute URLs

Fixes #2424

Author: sindresorhus

source/core/options.ts

@@ -1757,7 +1757,7 @@ export default class Options {
 
 		// Detect if URL is already absolute (has a protocol/scheme)
 		const valueString = value.toString();
-		const isAbsolute = is.urlInstance(value) || /^[a-z][a-z\d+.-]*:/i.test(valueString);
+		const isAbsolute = is.urlInstance(value) || /^[a-z][a-z\d+.-]*:\/\//i.test(valueString);
 
 		// Only concatenate prefixUrl if the URL is relative
 		const urlString = isAbsolute ? valueString : `${this.prefixUrl as string}${valueString}`;

test/hooks.ts

@@ -664,6 +664,58 @@ test('setting absolute URL in hook does not concatenate with prefixUrl', withSer
 	t.is(body, 'changed');
 });
 
+test('allows colon in path segment with prefixUrl (CouchDB user URLs)', withServer, async (t, server, serverGot) => {
+	server.get('/_users/org.couchdb.user:[email protected]', (_request, response) => {
+		response.end('user document');
+	});
+
+	const client = serverGot.extend({
+		prefixUrl: `${server.url}/_users/`,
+	});
+
+	const {body} = await client.get('org.couchdb.user:[email protected]');
+	t.is(body, 'user document');
+});
+
+test('allows multiple colons in path with prefixUrl', withServer, async (t, server, serverGot) => {
+	server.get('/api/ns:type:id', (_request, response) => {
+		response.end('namespaced');
+	});
+
+	const client = serverGot.extend({
+		prefixUrl: `${server.url}/api/`,
+	});
+
+	const {body} = await client.get('ns:type:id');
+	t.is(body, 'namespaced');
+});
+
+test('allows mailto-like patterns in path with prefixUrl', withServer, async (t, server, serverGot) => {
+	server.get('/users/mailto:[email protected]', (_request, response) => {
+		response.end('email user');
+	});
+
+	const client = serverGot.extend({
+		prefixUrl: `${server.url}/users/`,
+	});
+
+	const {body} = await client.get('mailto:[email protected]');
+	t.is(body, 'email user');
+});
+
+test('allows URN-like patterns in path with prefixUrl', withServer, async (t, server, serverGot) => {
+	server.get('/resources/urn:isbn:123', (_request, response) => {
+		response.end('book');
+	});
+
+	const client = serverGot.extend({
+		prefixUrl: `${server.url}/resources/`,
+	});
+
+	const {body} = await client.get('urn:isbn:123');
+	t.is(body, 'book');
+});
+
 test('afterResponse is called with response', withServer, async (t, server, got) => {
 	server.get('/', echoHeaders);