[Instrumentation.Http] fall back in case of NET9. update unit tests

[TimothyMothra]

Fixes #2029

Net9 added support for Http semantic convention attributes here: dotnet/runtime#104251

Changes

• update Instrumentation.Http to not override tags set by NET9.
• update unit tests

Merge requirement checklist

• CONTRIBUTING guidelines followed (license requirements, nullable enabled, static analysis, etc.)
• Unit tests added/updated
• Appropriate CHANGELOG.md files updated for non-trivial changes
• Changes in public API reviewed (if applicable)

[codecov]

Codecov Report

Attention: Patch coverage is 84.61538% with 2 lines in your changes missing coverage. Please review.

Project coverage is 68.24%. Comparing base (71655ce) to head (292c915).
Report is 600 commits behind head on main.

Files with missing lines	Patch %	Lines
...ementation/HttpHandlerMetricsDiagnosticListener.cs	0.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2314      +/-   ##
==========================================
- Coverage   73.91%   68.24%   -5.67%     
==========================================
  Files         267      363      +96     
  Lines        9615    13816    +4201     
==========================================
+ Hits         7107     9429    +2322     
- Misses       2508     4387    +1879     

Flag	Coverage Δ
unittests-Instrumentation.Http	74.12% <84.61%> (?)

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

Files with missing lines	Coverage Δ
...tp/Implementation/HttpHandlerDiagnosticListener.cs	72.22% <100.00%> (ø)
...ementation/HttpHandlerMetricsDiagnosticListener.cs	0.00% <0.00%> (ø)

... and 383 files with indirect coverage changes

[TimothyMothra]

I need to add NET9 to the CI. I'm checking offline to confirm if this is the correct way to make that change.

[xiang17]

After #2313 gets merged, net9.0 would run for all jobs by default. You won't need to configure for this individual job.

[rajkumar-rangaraj]

@TimothyMothra As we discussed offline, please evaluate if we need to cover the .NET 9.0 gaps for url.full with URL fragment and http.request.method_original case sensitivity.

[TimothyMothra]

I opened a tracking issue in Runtime: dotnet/runtime#109847

[alanwest]

PR looks pretty good, but looks like there's some build issues.

[TimothyMothra]

PR looks pretty good

Thanks!

but looks like there's some build issues.

Sigh. Visual Studio keeps saving files with Windows line endings (CR LF), but this repo expects Unix line endings (LF).
I haven't found a way to force Visual Studio to stop changing things. 🤦‍♂️