hono 3.1.3 → 3.1.4

package/dist/cjs/request.js

@@ -49,18 +49,10 @@ class HonoRequest {
     return null;
   }
   query(key) {
-    const queryString = (0, import_url.getQueryStringFromURL)(this.url);
-    const result = (0, import_url.getQueryParam)(queryString, key);
-    if (result === null)
-      return void 0;
-    return result;
+    return (0, import_url.getQueryParam)(this.url, key);
   }
   queries(key) {
-    const queryString = (0, import_url.getQueryStringFromURL)(this.url);
-    const result = (0, import_url.getQueryParams)(queryString, key);
-    if (result === null)
-      return void 0;
-    return result;
+    return (0, import_url.getQueryParams)(this.url, key);
   }
   header(name) {
     const headerData = {};

package/dist/cjs/utils/url.js

@@ -23,7 +23,6 @@ __export(url_exports, {
   getPattern: () => getPattern,
   getQueryParam: () => getQueryParam,
   getQueryParams: () => getQueryParams,
-  getQueryStringFromURL: () => getQueryStringFromURL,
   mergePath: () => mergePath,
   splitPath: () => splitPath,
   splitRoutingPath: () => splitRoutingPath
@@ -92,11 +91,6 @@ const getPathFromURL = (url, strict = true) => {
   }
   return result;
 };
-const getQueryStringFromURL = (url) => {
-  const queryIndex = url.indexOf("?", 8);
-  const result = queryIndex !== -1 ? url.slice(queryIndex + 1) : "";
-  return result;
-};
 const mergePath = (...paths) => {
   let p = "";
   let endsWithSlash = false;
@@ -127,48 +121,71 @@ const checkOptionalParameter = (path) => {
   const optional = base + match[2];
   return [base === "" ? "/" : base.replace(/\/$/, ""), optional];
 };
-const getQueryParam = (queryString, key) => {
-  const results = {};
-  while (true) {
-    const andIndex = queryString.indexOf("&");
-    let strings = "";
-    if (andIndex === -1) {
-      strings = queryString;
-    } else {
-      strings = queryString.substring(0, andIndex);
+const _decodeURI = (value) => {
+  if (!/[%+]/.test(value)) {
+    return value;
+  }
+  if (value.includes("+")) {
+    value = value.replace(/\+/g, " ");
+  }
+  return value.includes("%") ? decodeURIComponent(value) : value;
+};
+const _getQueryParam = (url, key, multiple) => {
+  let encoded;
+  if (!multiple && key && !/[%+]/.test(key)) {
+    let keyIndex2 = url.indexOf(`?${key}`, 8);
+    if (keyIndex2 === -1) {
+      keyIndex2 = url.indexOf(`&${key}`, 8);
     }
-    const eqIndex = strings.indexOf("=");
-    if (eqIndex !== -1) {
-      const v = strings.substring(eqIndex + 1);
-      const k = strings.substring(0, eqIndex);
-      if (key === k) {
-        return /\%/.test(v) ? decodeURIComponent(v) : v;
-      } else {
-        results[k] || (results[k] = v);
+    while (keyIndex2 !== -1) {
+      const trailingKeyCode = url.charCodeAt(keyIndex2 + key.length + 1);
+      if (trailingKeyCode === 61) {
+        const valueIndex = keyIndex2 + key.length + 2;
+        const endIndex = url.indexOf("&", valueIndex);
+        const value = url.slice(valueIndex, endIndex === -1 ? void 0 : endIndex);
+        return _decodeURI(value);
+      } else if (trailingKeyCode == 38 || isNaN(trailingKeyCode)) {
+        return "";
       }
-    } else if (strings === key) {
-      return "";
+      keyIndex2 = url.indexOf(`&${key}`, keyIndex2);
+    }
+    encoded = /[%+]/.test(url);
+    if (!encoded) {
+      return void 0;
     }
-    if (andIndex === -1)
-      break;
-    queryString = queryString.substring(andIndex + 1, queryString.length);
   }
-  if (key)
-    return null;
-  return results;
-};
-const getQueryParams = (queryString, key) => {
   const results = {};
-  for (const strings of queryString.split("&")) {
-    let [k, v] = strings.split("=");
-    if (v === void 0)
-      v = "";
-    results[k] || (results[k] = []);
-    results[k].push(v.indexOf("%") !== -1 ? decodeURIComponent(v) : v);
+  encoded ?? (encoded = /[%+]/.test(url));
+  let keyIndex = url.indexOf("?", 8);
+  while (keyIndex !== -1) {
+    const valueIndex = url.indexOf("=", keyIndex);
+    let name = url.slice(keyIndex + 1, valueIndex === -1 ? void 0 : valueIndex);
+    if (encoded) {
+      name = _decodeURI(name);
+    }
+    let value;
+    if (valueIndex === -1) {
+      keyIndex = -1;
+      value = "";
+    } else {
+      keyIndex = url.indexOf("&", valueIndex);
+      value = url.slice(valueIndex + 1, keyIndex === -1 ? void 0 : keyIndex);
+      if (encoded) {
+        value = _decodeURI(value);
+      }
+    }
+    if (multiple) {
+      ;
+      (results[name] ?? (results[name] = [])).push(value);
+    } else {
+      results[name] ?? (results[name] = value);
+    }
   }
-  if (key)
-    return results[key] ? results[key] : null;
-  return results;
+  return key ? results[key] : results;
+};
+const getQueryParam = _getQueryParam;
+const getQueryParams = (url, key) => {
+  return _getQueryParam(url, key, true);
 };
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {
@@ -177,7 +194,6 @@ const getQueryParams = (queryString, key) => {
   getPattern,
   getQueryParam,
   getQueryParams,
-  getQueryStringFromURL,
   mergePath,
   splitPath,
   splitRoutingPath

package/dist/request.js

@@ -1,7 +1,7 @@
 // src/request.ts
 import { parseBody } from "./utils/body.js";
 import { parse } from "./utils/cookie.js";
-import { getQueryStringFromURL, getQueryParam, getQueryParams } from "./utils/url.js";
+import { getQueryParam, getQueryParams } from "./utils/url.js";
 var HonoRequest = class {
   constructor(request, path = "/", paramData) {
     this.raw = request;
@@ -27,18 +27,10 @@ var HonoRequest = class {
     return null;
   }
   query(key) {
-    const queryString = getQueryStringFromURL(this.url);
-    const result = getQueryParam(queryString, key);
-    if (result === null)
-      return void 0;
-    return result;
+    return getQueryParam(this.url, key);
   }
   queries(key) {
-    const queryString = getQueryStringFromURL(this.url);
-    const result = getQueryParams(queryString, key);
-    if (result === null)
-      return void 0;
-    return result;
+    return getQueryParams(this.url, key);
   }
   header(name) {
     const headerData = {};

package/dist/types/utils/url.d.ts

@@ -3,8 +3,7 @@ export declare const splitPath: (path: string) => string[];
 export declare const splitRoutingPath: (path: string) => string[];
 export declare const getPattern: (label: string) => Pattern | null;
 export declare const getPathFromURL: (url: string, strict?: boolean) => string;
-export declare const getQueryStringFromURL: (url: string) => string;
 export declare const mergePath: (...paths: string[]) => string;
 export declare const checkOptionalParameter: (path: string) => string[] | null;
-export declare const getQueryParam: (queryString: string, key?: string) => string | null | Record<string, string>;
-export declare const getQueryParams: (queryString: string, key?: string) => string[] | null | Record<string, string[]>;
+export declare const getQueryParam: (url: string, key?: string) => string | undefined | Record<string, string>;
+export declare const getQueryParams: (url: string, key?: string) => string[] | undefined | Record<string, string[]>;

package/dist/utils/url.js

@@ -62,11 +62,6 @@ var getPathFromURL = (url, strict = true) => {
   }
   return result;
 };
-var getQueryStringFromURL = (url) => {
-  const queryIndex = url.indexOf("?", 8);
-  const result = queryIndex !== -1 ? url.slice(queryIndex + 1) : "";
-  return result;
-};
 var mergePath = (...paths) => {
   let p = "";
   let endsWithSlash = false;
@@ -97,48 +92,71 @@ var checkOptionalParameter = (path) => {
   const optional = base + match[2];
   return [base === "" ? "/" : base.replace(/\/$/, ""), optional];
 };
-var getQueryParam = (queryString, key) => {
-  const results = {};
-  while (true) {
-    const andIndex = queryString.indexOf("&");
-    let strings = "";
-    if (andIndex === -1) {
-      strings = queryString;
-    } else {
-      strings = queryString.substring(0, andIndex);
+var _decodeURI = (value) => {
+  if (!/[%+]/.test(value)) {
+    return value;
+  }
+  if (value.includes("+")) {
+    value = value.replace(/\+/g, " ");
+  }
+  return value.includes("%") ? decodeURIComponent(value) : value;
+};
+var _getQueryParam = (url, key, multiple) => {
+  let encoded;
+  if (!multiple && key && !/[%+]/.test(key)) {
+    let keyIndex2 = url.indexOf(`?${key}`, 8);
+    if (keyIndex2 === -1) {
+      keyIndex2 = url.indexOf(`&${key}`, 8);
     }
-    const eqIndex = strings.indexOf("=");
-    if (eqIndex !== -1) {
-      const v = strings.substring(eqIndex + 1);
-      const k = strings.substring(0, eqIndex);
-      if (key === k) {
-        return /\%/.test(v) ? decodeURIComponent(v) : v;
-      } else {
-        results[k] || (results[k] = v);
+    while (keyIndex2 !== -1) {
+      const trailingKeyCode = url.charCodeAt(keyIndex2 + key.length + 1);
+      if (trailingKeyCode === 61) {
+        const valueIndex = keyIndex2 + key.length + 2;
+        const endIndex = url.indexOf("&", valueIndex);
+        const value = url.slice(valueIndex, endIndex === -1 ? void 0 : endIndex);
+        return _decodeURI(value);
+      } else if (trailingKeyCode == 38 || isNaN(trailingKeyCode)) {
+        return "";
       }
-    } else if (strings === key) {
-      return "";
+      keyIndex2 = url.indexOf(`&${key}`, keyIndex2);
+    }
+    encoded = /[%+]/.test(url);
+    if (!encoded) {
+      return void 0;
     }
-    if (andIndex === -1)
-      break;
-    queryString = queryString.substring(andIndex + 1, queryString.length);
   }
-  if (key)
-    return null;
-  return results;
-};
-var getQueryParams = (queryString, key) => {
   const results = {};
-  for (const strings of queryString.split("&")) {
-    let [k, v] = strings.split("=");
-    if (v === void 0)
-      v = "";
-    results[k] || (results[k] = []);
-    results[k].push(v.indexOf("%") !== -1 ? decodeURIComponent(v) : v);
+  encoded ?? (encoded = /[%+]/.test(url));
+  let keyIndex = url.indexOf("?", 8);
+  while (keyIndex !== -1) {
+    const valueIndex = url.indexOf("=", keyIndex);
+    let name = url.slice(keyIndex + 1, valueIndex === -1 ? void 0 : valueIndex);
+    if (encoded) {
+      name = _decodeURI(name);
+    }
+    let value;
+    if (valueIndex === -1) {
+      keyIndex = -1;
+      value = "";
+    } else {
+      keyIndex = url.indexOf("&", valueIndex);
+      value = url.slice(valueIndex + 1, keyIndex === -1 ? void 0 : keyIndex);
+      if (encoded) {
+        value = _decodeURI(value);
+      }
+    }
+    if (multiple) {
+      ;
+      (results[name] ?? (results[name] = [])).push(value);
+    } else {
+      results[name] ?? (results[name] = value);
+    }
   }
-  if (key)
-    return results[key] ? results[key] : null;
-  return results;
+  return key ? results[key] : results;
+};
+var getQueryParam = _getQueryParam;
+var getQueryParams = (url, key) => {
+  return _getQueryParam(url, key, true);
 };
 export {
   checkOptionalParameter,
@@ -146,7 +164,6 @@ export {
   getPattern,
   getQueryParam,
   getQueryParams,
-  getQueryStringFromURL,
   mergePath,
   splitPath,
   splitRoutingPath

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hono",
-  "version": "3.1.3",
+  "version": "3.1.4",
   "description": "Ultrafast web framework for the Edge",
   "main": "dist/cjs/index.js",
   "type": "module",