[tammy-baylis-swi]

An agent helped me find that context cleanup (explicit token detach) by APM Python HttpSampler and UAMS detector may not succeed and persist, if exception raised by requests.get. This adds try-finally and updates unit tests.

[Copilot]

🟢 Approval recommended

No active review issues remain for the HTTP-request context-cleanup changes in the sampler and UAMS resource detector.

No open issues 🎉

What changed in this PR

This PR prevents instrumentation suppression from persisting after failed HTTP requests in the SolarWinds APM sampler and UAMS resource detector.

Changes:

• Wraps both HTTP requests in try/finally so context tokens are always detached.
• Adds regression tests verifying cleanup after connection errors while preserving existing error behavior.

File	Description
tests/​unit/​test_uams.py	Verifies context cleanup and an empty result after a connection error.
tests/​unit/​test_oboe/​test_http_sampler.py	Verifies context cleanup while allowing the connection error to propagate.
solarwinds_apm/​uams.py	Ensures context detachment when the UAMS request fails.
solarwinds_apm/​oboe/​http_sampler.py	Ensures context detachment when the collector request fails.

_{Review effort: Balanced}