fix(manifest-server): serve /health and /capabilities without 307 redirect

airbyte_cdk/manifest_server/routers/capabilities.py

@@ -11,6 +11,7 @@
 )
 
 
+@router.get("", include_in_schema=False)
 @router.get("/", operation_id="getCapabilities")
 def get_capabilities() -> CapabilitiesResponse:
     """

airbyte_cdk/manifest_server/routers/health.py

@@ -8,6 +8,7 @@
 )
 
 
+@router.get("", include_in_schema=False)
 @router.get("/")
 def health() -> Dict[str, str]:
     return {"status": "ok"}

unit_tests/manifest_server/routers/test_capabilities.py

@@ -5,7 +5,7 @@
 
 from airbyte_cdk.manifest_server.app import app
 
-client = TestClient(app)
+client = TestClient(app, follow_redirects=False)
 
 
 class TestCapabilities:
@@ -16,6 +16,11 @@ def test_capabilities_endpoint_exists(self):
         response = client.get("/capabilities/")
         assert response.status_code == 200
 
+    def test_capabilities_endpoint_without_trailing_slash(self):
+        """Test that the capabilities endpoint does not redirect without a trailing slash."""
+        response = client.get("/capabilities")
+        assert response.status_code == 200
+
     def test_capabilities_custom_code_execution_false_by_default(self):
         """Test that custom_code_execution is false by default when env var is not set."""
         with patch.dict(os.environ, {}, clear=True):

unit_tests/manifest_server/routers/test_health.py

@@ -0,0 +1,18 @@
+from fastapi.testclient import TestClient
+
+from airbyte_cdk.manifest_server.app import app
+
+client = TestClient(app, follow_redirects=False)
+
+
+def test_health_endpoint_without_trailing_slash() -> None:
+    response = client.get("/health")
+
+    assert response.status_code == 200
+    assert response.json() == {"status": "ok"}
+
+
+def test_health_endpoint_with_trailing_slash() -> None:
+    response = client.get("/health/")
+
+    assert response.status_code == 200