fix: Dropped query params when obfuscating

[mrickard]

Changed http to use pathname instead of path, avoiding exposure of query params

How to Test

Test for this case has been added to tests/unit/instrumentation/http/http.test.js

Related Issues

Fixes #1718
Closes NR-141506

[bizob2828]

nice work, hopefully this will prevent regressions in future

[codecov]

Codecov Report

Merging #1719 (4f7e0fb) into main (20442c6) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1719   +/-   ##
=======================================
  Coverage   96.87%   96.87%           
=======================================
  Files         200      200           
  Lines       39181    39181           
  Branches       24       24           
=======================================
  Hits        37956    37956           
  Misses       1225     1225           

Flag	Coverage Δ
esm-unit-tests-14.x	47.80% <ø> (ø)
esm-unit-tests-16.x	92.11% <ø> (ø)
esm-unit-tests-18.x	92.11% <ø> (ø)
integration-tests-14.x	78.98% <100.00%> (ø)
integration-tests-16.x	79.07% <100.00%> (+<0.01%)	⬆️
integration-tests-18.x	79.06% <100.00%> (ø)
unit-tests-14.x	91.41% <100.00%> (+<0.01%)	⬆️
unit-tests-16.x	91.47% <100.00%> (+<0.01%)	⬆️
unit-tests-18.x	91.45% <100.00%> (+<0.01%)	⬆️
versioned-tests-14.x	75.70% <100.00%> (-0.06%)	⬇️
versioned-tests-16.x	77.03% <100.00%> (-0.05%)	⬇️
versioned-tests-18.x	77.03% <100.00%> (-0.05%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
lib/instrumentation/core/http.js	90.84% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more