Fix URL construction for domain with port

- Updated URL construction logic to check for '://' to distinguish between real protocols and domain:port format
- Added test for domain with port (example.com:8080) to ensure https:// is prepended correctly
- All 28 tests pass

Co-authored-by: sergioescalera <[email protected]>

Author: Copilot

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

@@ -102,14 +102,15 @@ def export(self, spans: Sequence[ReadableSpan]) -> SpanExportResult:
                     else f"/maven/agent365/agents/{agent_id}/traces"
                 )
 
-                # Construct URL - if endpoint already has a scheme (http:// or https://), use it as-is
+                # 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
                 parsed = urlparse(endpoint)
-                if parsed.scheme:
+                if parsed.scheme and "://" in endpoint:
                     # Endpoint is a full URL, append path
                     url = f"{endpoint}{endpoint_path}?api-version=1"
                 else:
-                    # Endpoint is just a domain, prepend https://
+                    # Endpoint is just a domain (possibly with port), prepend https://
                     url = f"https://{endpoint}{endpoint_path}?api-version=1"
 
                 # Debug: Log endpoint being used

tests/observability/core/test_agent365_exporter.py

@@ -577,6 +577,41 @@ def test_export_uses_valid_url_override_with_http(self):
                 # Verify PowerPlatformApiDiscovery was not called
                 mock_discovery_class.assert_not_called()
 
+    def test_export_uses_valid_domain_override_with_port(self):
+        """Test that domain override with port (no protocol) is accepted and https:// is prepended."""
+        # Arrange
+        os.environ["A365_OBSERVABILITY_DOMAIN_OVERRIDE"] = "example.com:8080"
+
+        # Create exporter after setting environment variable
+        exporter = _Agent365Exporter(
+            token_resolver=self.mock_token_resolver, cluster_category="test"
+        )
+
+        spans = [self._create_mock_span("test_span")]
+
+        # Mock the PowerPlatformApiDiscovery class (should NOT be called since override is valid)
+        with patch(
+            "microsoft_agents_a365.observability.core.exporters.agent365_exporter.PowerPlatformApiDiscovery"
+        ) as mock_discovery_class:
+            # Mock the _post_with_retries method
+            with patch.object(exporter, "_post_with_retries", return_value=True) as mock_post:
+                # Act
+                result = exporter.export(spans)
+
+                # Assert
+                self.assertEqual(result, SpanExportResult.SUCCESS)
+                mock_post.assert_called_once()
+
+                # Verify the call arguments - should prepend https:// to domain with port
+                args, kwargs = mock_post.call_args
+                url, body, headers = args
+
+                expected_url = "https://example.com:8080/maven/agent365/agents/test-agent-456/traces?api-version=1"
+                self.assertEqual(url, expected_url)
+
+                # Verify PowerPlatformApiDiscovery was not called
+                mock_discovery_class.assert_not_called()
+
     def test_export_ignores_invalid_domain_with_protocol(self):
         """Test that domain override with invalid protocol is ignored."""
         # Arrange