chore: use url.URL instead of url.parse (#1340)

The url.parse function is deprecated

Author: yuxizhe

lib/instrumentation/express-utils.js

@@ -1,13 +1,14 @@
 'use strict'
 
+var symbols = require('../symbols')
+var parsers = require('../parsers')
+
 var parseUrl
 try {
   parseUrl = require('parseurl')
 } catch (e) {
-  const url = require('url')
-  parseUrl = req => url.parse(req.url)
+  parseUrl = req => parsers.parseUrl(req.url)
 }
-var symbols = require('../symbols')
 
 function normalizeSlash (value) {
   return value[0] === '/' ? value : '/' + value

lib/instrumentation/http-shared.js

@@ -4,6 +4,8 @@ var url = require('url')
 
 var endOfStream = require('end-of-stream')
 
+var parsers = require('../parsers')
+
 const transactionForResponse = new WeakMap()
 exports.transactionForResponse = transactionForResponse
 
@@ -80,15 +82,27 @@ function isRequestBlacklisted (agent, req) {
   return false
 }
 
+function formatURL (item) {
+  return {
+    href: item.href,
+    pathname: item.pathname,
+    path: item.pathname + (item.search || ''),
+    protocol: item.protocol,
+    host: item.host,
+    port: item.port,
+    hostname: item.hostname,
+    hash: item.hash,
+    search: item.search
+  }
+}
+
 // NOTE: This will also stringify and parse URL instances
 // to a format which can be mixed into the options object.
 function ensureUrl (v) {
   if (typeof v === 'string') {
-    return url.parse(v)
-  } else if (v instanceof url.Url) {
-    return v
-  } else if (url.URL && v instanceof url.URL) { // check for url.URL because it wasn't added until Node.js 6.13.0
-    return url.parse(v.toString())
+    return formatURL(parsers.parseUrl(v))
+  } else if (url.URL && v instanceof url.URL) {
+    return formatURL(v)
   } else {
     return v
   }
@@ -142,7 +156,7 @@ exports.traceOutgoingRequest = function (agent, moduleName, method) {
 
       ins.bindEmitter(req)
 
-      span.name = req.method + ' ' + req._headers.host + url.parse(req.path).pathname
+      span.name = req.method + ' ' + req._headers.host + parsers.parseUrl(req.path).pathname
 
       // TODO: Research if it's possible to add this to the prototype instead.
       // Or if it's somehow preferable to listen for when a `response` listener

lib/instrumentation/modules/http2.js

@@ -1,11 +1,10 @@
 'use strict'
 
-var url = require('url')
-
 var eos = require('end-of-stream')
 
 var shimmer = require('../shimmer')
 var symbols = require('../../symbols')
+var parsers = require('../../parsers')
 
 module.exports = function (http2, agent, { enabled }) {
   if (!enabled) return http2
@@ -155,7 +154,8 @@ module.exports = function (http2, agent, { enabled }) {
 
       ins.bindEmitter(req)
 
-      var path = url.parse(headers[':path']).pathname
+      var urlObj = parsers.parseUrl(headers[':path'])
+      var path = urlObj.pathname
       span.name = headers[':method'] + ' ' + host + path
 
       req.on('end', () => {

lib/parsers.js

@@ -1,6 +1,7 @@
 'use strict'
 
 var util = require('util')
+var url = require('url')
 
 var afterAll = require('after-all-results')
 var basicAuth = require('basic-auth')
@@ -215,6 +216,10 @@ exports.parseCallsite = function (callsite, isError, agent, cb) {
   })
 }
 
+exports.parseUrl = function (urlStr) {
+  return url.URL ? new url.URL(urlStr, 'relative:///') : url.parse(urlStr)
+}
+
 // Default `culprit` to the top of the stack or the highest non `library_frame`
 // frame if such exists
 function getCulprit (frames) {