chore: Removed unused transaction method (#2986)

Author: jsumners-nr

lib/transaction/index.js

@@ -15,7 +15,6 @@ const props = require('../util/properties')
 const Timer = require('../timer')
 const Trace = require('./trace')
 const synthetics = require('../synthetics')
-const url = require('url')
 const urltils = require('../util/urltils')
 const TraceContext = require('./tracecontext').TraceContext
 const Logs = require('./logs')
@@ -609,49 +608,6 @@ Transaction.prototype.getFullName = function getFullName() {
   return fullName
 }
 
-/**
- * Returns the full URL of the transaction with query, search, or hash portions
- * removed. This is only applicable for web transactions.
- *
- * Caches to ._scrubbedUrl, pulls in from .parsedUrl if it is available,
- * otherwise it will parse .url, store it on .parsedUrl, then scrub the URL and
- * store it in the cache.
- *
- * @returns {(string|undefined)} Returns a string or undefined.
- */
-Transaction.prototype.getScrubbedUrl = function getScrubbedUrl() {
-  if (!this.isWeb()) {
-    return
-  }
-  if (this._scrubbedUrl) {
-    return this._scrubbedUrl
-  }
-
-  // If we don't have a parsedUrl, lets populate it from .url
-  if (!this.parsedUrl) {
-    // At time of writing .url should always be set by the time we get here
-    // because that is what .isWeb() checks against. In the future it may be
-    // instead checking a enum or other property so guard ourselves just in
-    // case.
-    if (!this.url) {
-      return
-    }
-
-    this.parsedUrl = url.parse(this.url)
-  }
-
-  const scrubbedParsedUrl = Object.assign(Object.create(null), this.parsedUrl)
-  scrubbedParsedUrl.search = null
-  scrubbedParsedUrl.query = null
-  scrubbedParsedUrl.href = null
-  scrubbedParsedUrl.path = null
-  scrubbedParsedUrl.hash = null
-
-  this._scrubbedUrl = url.format(scrubbedParsedUrl)
-
-  return this._scrubbedUrl
-}
-
 /**
  * The instrumentation associates metrics with the different kinds of trace
  * segments. The metrics recorders are dependent on the transaction name to