fix(authentication-azure): record is_url_valid as True, not pickle's TRUE opcode

[Max Azatian (HardMax71)]

Overview

AzureIdentityAccessTokenProvider imported TRUE from pickle and set the com.microsoft.kiota.authentication.is_url_valid span attribute to it once the URL passed validation. pickle.TRUE is the protocol 0 opcode b"I01\n", not a boolean, so the attribute never recorded True. It looks like an editor auto-import picked it up when observability was added in 2023.

What tracing backends get depends on the OpenTelemetry SDK version:

• Before 1.45.0 the SDK decodes bytes, so the attribute reads "I01\n".
• Since 1.45.0 (Expand attribute value type to support complex values everywhere and cleanup surrounding code open-telemetry/opentelemetry-python#5266) the SDK keeps bytes as they are. OTLP exports them as bytes_value, and ConsoleSpanExporter fails in span.to_json() with "Object of type bytes is not JSON serializable". The whole get_authorization_token span is dropped and only "Exception while exporting Span." gets logged.

The package allows opentelemetry-sdk>=1.27.0, so a fresh install today gets the second case. This sets a real True, which matches the False on the failure paths and what kiota-dotnet and kiota-typescript record.

Related Issue

None filed. I found it while typing this package for #722.

Notes

#785 touches the same file, and the two merge cleanly.

Testing Instructions

• The new test records spans with an in-memory exporter, the same way the http package's span_exporter fixture does, and checks that the attribute is True. On main it fails with assert b'I01\n' is True.
• yapf -dr, isort, pylint --disable=W, mypy and pytest pass in packages/authentication/azure.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[Vincent Biret (baywet)]

Thanks for the contribution!

[Copilot]

🟢 Approval recommended

The focused fix is correct and has adequate regression coverage.

0 open findings

What changed in this PR

Fixes Azure authentication tracing to record URL validity as a boolean.

Changes:

• Replaces pickle.TRUE with Python’s True.
• Adds regression coverage for the span attribute.

File	Description
azure_identity_access_token_provider.py	Records valid URL telemetry correctly.
test_azure_identity_access_token_provider.py	Verifies the attribute is boolean True.

🧠 Review effort: Balanced

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.