fix(instrumentation-http): include query params in http.target

[kobi-co]

Which problem is this PR solving?

According to the spec, http.target attribute should include path + query params, current implementation sets the path without query params.

Attribute	Type	Description	Examples	Requirement Level
http.target	string	The full request target as passed in a HTTP request line or equivalent.	/path/12314/?q=ddds	Required

Short description of the changes

Fix http.target to use urlObject.path and not urlObject.pathname which is the value the includes the query params.

Type of change

Bug fix

How Has This Been Tested?

Created a new unit test to check http.target attribute value.

[codecov]

Codecov Report

Merging #3646 (89d3aa1) into main (0d373bd) will increase coverage by 0.20%.
The diff coverage is 100.00%.

❗ Current head 89d3aa1 differs from pull request most recent head f7365e2. Consider uploading reports for the commit f7365e2 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3646      +/-   ##
==========================================
+ Coverage   93.52%   93.73%   +0.20%     
==========================================
  Files         271      274       +3     
  Lines        7496     8058     +562     
  Branches     1521     1670     +149     
==========================================
+ Hits         7011     7553     +542     
- Misses        485      505      +20     

Impacted Files	Coverage Δ
...es/opentelemetry-instrumentation-http/src/utils.ts	99.18% <100.00%> (ø)
...emetry-core/src/platform/node/RandomIdGenerator.ts	87.50% <0.00%> (-6.25%)	⬇️
...y-instrumentation-http/src/enums/AttributeNames.ts	100.00% <0.00%> (ø)
...ges/opentelemetry-instrumentation-http/src/http.ts	94.26% <0.00%> (ø)
...-trace-base/src/platform/node/RandomIdGenerator.ts	93.75% <0.00%> (+6.25%)	⬆️