[http] fix: use url.URL (#960)

url.URL from the standard library should be used instead of the global URL which comes from the dom lib. Signed-off-by: Naseem <naseem@transit.app>
open-telemetry · Apr 16, 2020 · 1959930 · 1959930
1 parent 7412d9b
commit 1959930
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/packages/opentelemetry-plugin-http/src/http.ts b/packages/opentelemetry-plugin-http/src/http.ts
@@ -145,7 +145,7 @@ export class HttpPlugin extends BasePlugin<Http> {
 
   protected _getPatchOutgoingGetFunction(
     clientRequest: (
-      options: RequestOptions | string | URL,
+      options: RequestOptions | string | url.URL,
       ...args: HttpRequestArgs
     ) => ClientRequest
   ) {
@@ -161,7 +161,7 @@ export class HttpPlugin extends BasePlugin<Http> {
       // https://nodejs.org/dist/latest/docs/api/http.html#http_http_get_options_callback
       // https://github.com/googleapis/cloud-trace-nodejs/blob/master/src/plugins/plugin-http.ts#L198
       return function outgoingGetRequest<
-        T extends RequestOptions | string | URL
+        T extends RequestOptions | string | url.URL
       >(options: T, ...args: HttpRequestArgs): ClientRequest {
         const req = clientRequest(options, ...args);
         req.end();