url: replace url-parser with ada

PR-URL: nodejs#46410 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com> Reviewed-By: Robert Nagy <ronagy@icloud.com>
anonrig · Apr 11, 2023 · e5385e5 · e5385e5
1 parent f5efd65
commit e5385e5
Show file tree

Hide file tree

Showing 17 changed files with 376 additions and 2,961 deletions.
diff --git a/lib/internal/url.js b/lib/internal/url.js
diff --git a/node.gyp b/node.gyp
@@ -542,7 +542,6 @@
         'src/node_trace_events.cc',
         'src/node_types.cc',
         'src/node_url.cc',
-        'src/node_url_tables.cc',
         'src/node_util.cc',
         'src/node_v8.cc',
         'src/node_wasi.cc',
@@ -1256,7 +1255,6 @@
         'test/cctest/test_sockaddr.cc',
         'test/cctest/test_traced_value.cc',
         'test/cctest/test_util.cc',
-        'test/cctest/test_url.cc',
       ],
 
       'conditions': [

diff --git a/src/crypto/crypto_common.cc b/src/crypto/crypto_common.cc
@@ -1,13 +1,12 @@
+#include "crypto/crypto_common.h"
 #include "base_object-inl.h"
 #include "env-inl.h"
+#include "memory_tracker-inl.h"
+#include "node.h"
 #include "node_buffer.h"
 #include "node_crypto.h"
-#include "crypto/crypto_common.h"
-#include "node.h"
 #include "node_internals.h"
-#include "node_url.h"
 #include "string_bytes.h"
-#include "memory_tracker-inl.h"
 #include "v8.h"
 
 #include <openssl/ec.h>

diff --git a/src/inspector_agent.cc b/src/inspector_agent.cc
@@ -647,8 +647,9 @@ class NodeInspectorClient : public V8InspectorClient {
         protocol::StringUtil::StringViewToUtf8(resource_name_view);
     if (!IsFilePath(resource_name))
       return nullptr;
-    node::url::URL url = node::url::URL::FromFilePath(resource_name);
-    return Utf8ToStringView(url.href());
+
+    std::string url = node::url::FromFilePath(resource_name);
+    return Utf8ToStringView(url);
   }
 
   node::Environment* env_;

diff --git a/src/module_wrap.cc b/src/module_wrap.cc
@@ -6,7 +6,6 @@
 #include "node_errors.h"
 #include "node_internals.h"
 #include "node_process-inl.h"
-#include "node_url.h"
 #include "node_watchdog.h"
 #include "util-inl.h"
 
@@ -20,8 +19,6 @@ namespace loader {
 using errors::TryCatchScope;
 
 using node::contextify::ContextifyContext;
-using node::url::URL;
-using node::url::URL_FLAGS_FAILED;
 using v8::Array;
 using v8::ArrayBufferView;
 using v8::Context;

diff --git a/src/node_api.cc b/src/node_api.cc
@@ -657,7 +657,7 @@ void napi_module_register_by_symbol(v8::Local<v8::Object> exports,
     // a file system path.
     // TODO(gabrielschulhof): Pass the `filename` through unchanged if/when we
     // receive it as a URL already.
-    module_filename = node::url::URL::FromFilePath(filename.ToString()).href();
+    module_filename = node::url::FromFilePath(filename.ToString());
   }
 
   // Create a new napi_env for this specific module.

diff --git a/src/node_metadata.cc b/src/node_metadata.cc
@@ -1,5 +1,6 @@
 #include "node_metadata.h"
 #include "acorn_version.h"
+#include "ada.h"
 #include "ares.h"
 #include "brotli/encode.h"
 #include "llhttp.h"
@@ -115,6 +116,7 @@ Metadata::Versions::Versions() {
 #endif
 
   simdutf = SIMDUTF_VERSION;
+  ada = ADA_VERSION;
 }
 
 Metadata::Release::Release() : name(NODE_RELEASE) {

diff --git a/src/node_metadata.h b/src/node_metadata.h
@@ -47,6 +47,7 @@ namespace node {
   V(uvwasi)                                                                    \
   V(acorn)                                                                     \
   V(simdutf)                                                                   \
+  V(ada)                                                                       \
   NODE_VERSIONS_KEY_UNDICI(V)
 
 #if HAVE_OPENSSL