fixup! url: use resolved path to convert UNC paths to URL

nodejs · Dec 23, 2024 · 85afa77 · 85afa77
1 parent 6708d1c
commit 85afa77
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/lib/internal/url.js b/lib/internal/url.js
@@ -1512,8 +1512,11 @@ function fileURLToPath(path, options = kEmptyObject) {
 
 function pathToFileURL(filepath, options = kEmptyObject) {
   const windows = options?.windows ?? isWindows;
-  let resolved = windows ? path.win32.resolve(filepath) : path.posix.resolve(filepath);
-  if (windows && StringPrototypeStartsWith(resolved, '\\\\')) {
+  const isUNC = windows && StringPrototypeStartsWith(filepath, '\\\\');
+  let resolved = isUNC ?
+    filepath :
+    (windows ? path.win32.resolve(filepath) : path.posix.resolve(filepath));
+  if (isUNC || windows && StringPrototypeStartsWith(resolved, '\\\\')) {
     // UNC path format: \\server\share\resource
     // Handle extended UNC path and standard UNC path
     // "\\?\UNC\" path prefix should be ignored.