fix: add node custom inspect for URL.searchParams (#12763)

fixes #11482 Since URL.searchParams is an object too, we needed to make the tracked property loggable, similar to the URL object itself. This prevents the crashing when logging it in the server load.
sveltejs · Oct 7, 2024 · 2292170 · 2292170
1 parent 3591411
commit 2292170
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/.changeset/rude-shirts-raise.md b/.changeset/rude-shirts-raise.md
@@ -0,0 +1,5 @@
+---
+'@sveltejs/kit': patch
+---
+
+fix: prevent crash when logging URL search params in a server load function
diff --git a/packages/kit/src/utils/url.js b/packages/kit/src/utils/url.js
@@ -144,6 +144,11 @@ export function make_trackable(url, callback, search_params_callback) {
 		tracked[Symbol.for('nodejs.util.inspect.custom')] = (depth, opts, inspect) => {
 			return inspect(url, opts);
 		};
+
+		// @ts-ignore
+		tracked.searchParams[Symbol.for('nodejs.util.inspect.custom')] = (depth, opts, inspect) => {
+			return inspect(url.searchParams, opts);
+		};
 	}
 
 	if (DEV || !BROWSER) {