fetch: improve performance of urlHasHttpsScheme (#3094)

Author: Uzlopak

benchmarks/fetch/url-has-https-scheme.mjs

@@ -0,0 +1,22 @@
+import { bench, run } from 'mitata'
+import { urlHasHttpsScheme } from '../../lib/web/fetch/util.js'
+
+const httpString = 'http://example.com'
+const httpObject = { protocol: 'http:' }
+const httpsString = 'https://example.com'
+const httpsObject = { protocol: 'https:' }
+
+bench('urlHasHttpsScheme "http:" String', () => {
+  urlHasHttpsScheme(httpString)
+})
+bench('urlHasHttpsScheme "https:" String', () => {
+  urlHasHttpsScheme(httpsString)
+})
+bench('urlHasHttpsScheme "http:" Object', () => {
+  urlHasHttpsScheme(httpObject)
+})
+bench('urlHasHttpsScheme "https:" Object', () => {
+  urlHasHttpsScheme(httpsObject)
+})
+
+await run()

lib/web/fetch/util.js

@@ -1168,13 +1168,21 @@ function urlIsLocal (url) {
 
 /**
  * @param {string|URL} url
+ * @returns {boolean}
  */
 function urlHasHttpsScheme (url) {
-  if (typeof url === 'string') {
-    return url.startsWith('https:')
-  }
-
-  return url.protocol === 'https:'
+  return (
+    (
+      typeof url === 'string' &&
+      url[5] === ':' &&
+      url[0] === 'h' &&
+      url[1] === 't' &&
+      url[2] === 't' &&
+      url[3] === 'p' &&
+      url[4] === 's'
+    ) ||
+    url.protocol === 'https:'
+  )
 }
 
 /**

test/fetch/util.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const { test } = require('node:test')
+const { describe, test } = require('node:test')
 const assert = require('node:assert')
 const { tspl } = require('@matteo.collina/tspl')
 const util = require('../../lib/web/fetch/util')
@@ -338,3 +338,20 @@ test('parseMetadata', async (t) => {
     ])
   })
 })
+
+describe('urlHasHttpsScheme', () => {
+  const { urlHasHttpsScheme } = util
+
+  test('should return false for http url', () => {
+    assert.strictEqual(urlHasHttpsScheme('http://example.com'), false)
+  })
+  test('should return true for https url', () => {
+    assert.strictEqual(urlHasHttpsScheme('https://example.com'), true)
+  })
+  test('should return false for http object', () => {
+    assert.strictEqual(urlHasHttpsScheme({ protocol: 'http:' }), false)
+  })
+  test('should return true for https object', () => {
+    assert.strictEqual(urlHasHttpsScheme({ protocol: 'https:' }), true)
+  })
+})