Finish span with exception in SentryHttpMessageHandler

[Tyrrrz]

No description provided.

[bruno-garcia]

Could you please add a test that verifies this behavior and avoid regression?

[Tyrrrz]

Could you please add a test that verifies this behavior and avoid regression?

Good point. Done.

[codecov-commenter]

Codecov Report

Merging #1037 (ed9752f) into main (29d301a) will decrease coverage by 1.60%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##             main    #1037      +/-   ##
==========================================
- Coverage   81.44%   79.83%   -1.61%     
==========================================
  Files         190      190              
  Lines        5967     5967              
  Branches     1469     1469              
==========================================
- Hits         4860     4764      -96     
- Misses        685      788     +103     
+ Partials      422      415       -7     

Impacted Files	Coverage Δ
src/Sentry/SentryHttpMessageHandler.cs	79.31% <50.00%> (+6.89%)	⬆️
src/Sentry/PlatformAbstractions/FrameworkInfo.cs	0.00% <0.00%> (-100.00%)	⬇️
...ntry/PlatformAbstractions/RegistryKeyExtensions.cs	0.00% <0.00%> (-100.00%)	⬇️
...Sentry/PlatformAbstractions/FrameworkInfo.NetFx.cs	0.00% <0.00%> (-71.24%)	⬇️
...rmAbstractions/NetFxInstallationsEventProcessor.cs	4.54% <0.00%> (-68.19%)	⬇️
...ntry/Integrations/NetFxInstallationsIntegration.cs	33.33% <0.00%> (-66.67%)	⬇️
...ntry/PlatformAbstractions/FrameworkInstallation.cs	25.00% <0.00%> (-37.50%)	⬇️
src/Sentry/PlatformAbstractions/RuntimeInfo.cs	53.44% <0.00%> (-5.18%)	⬇️
...ry/Internal/Http/DefaultSentryHttpClientFactory.cs	58.33% <0.00%> (-4.17%)	⬇️
src/Sentry/Internal/AppDomainAdapter.cs	66.66% <0.00%> (ø)
... and 1 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 29d301a...ed9752f. Read the comment docs.