Add clarifying comments for urlparse edge case

- Added comments explaining why checking for '://' is necessary
- urlparse treats 'example.com:8080' as having scheme='example.com', so we need to check for '://' to distinguish between real protocols and domain:port format
- All 28 tests still passing

Co-authored-by: sergioescalera <8428450+sergioescalera@users.noreply.github.com>

Author: Copilot

libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/exporters/agent365_exporter.py

@@ -105,6 +105,7 @@ def export(self, spans: Sequence[ReadableSpan]) -> SpanExportResult:
                 # Construct URL - if endpoint has a scheme (http:// or https://), use it as-is
                 # Otherwise, prepend https://
                 # Note: Check for "://" to distinguish between real protocols and domain:port format
+                # (urlparse treats "example.com:8080" as having scheme="example.com")
                 parsed = urlparse(endpoint)
                 if parsed.scheme and "://" in endpoint:
                     # Endpoint is a full URL, append path

libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/exporters/utils.py

@@ -159,6 +159,8 @@ def get_validated_domain_override() -> str | None:
         parsed = urlparse(domain_override)
 
         # If scheme is present and looks like a protocol (contains //)
+        # Note: We check for "://" because urlparse treats "example.com:8080" as having
+        # scheme="example.com", but this is actually a domain with port, not a protocol.
         if parsed.scheme and "://" in domain_override:
             # Validate it's http or https
             if parsed.scheme not in ("http", "https"):