Merge pull request #19 from contextforge-org/ServeNoAuthFlag-9

Serve no auth flag 9

Author: gabe-l-hart

cforge/commands/server/serve.py

@@ -32,6 +32,7 @@ def serve(
     port: int = typer.Option(DEFAULT_PORT, "--port", help="Port to bind to"),
     reload: bool = typer.Option(False, "--reload", help="Enable auto-reload for development"),
     headless: bool = typer.Option(False, "--headless", help="Run without the admin UI"),
+    no_auth: bool = typer.Option(False, "--no-auth", help="Run without requiring authorization"),
     workers: int = typer.Option(1, "--workers", help="Number of worker processes"),
     log_level: str = typer.Option("info", "--log-level", help="Log level (debug, info, warning, error, critical)"),
 ) -> None:
@@ -42,6 +43,7 @@ def serve(
     set_serve_settings(
         mcpgateway_ui_enabled=not headless,
         mcpgateway_admin_api_enabled=not headless,
+        auth_required=not no_auth,
     )
     uvicorn.run(
         DEFAULT_APP,

tests/commands/server/test_serve.py

@@ -17,7 +17,7 @@
 
 # First-Party
 from cforge.commands.server.serve import serve
-from tests.conftest import get_open_port
+from tests.conftest import get_open_port, invoke_typer_command
 
 
 class TestServeCommand:
@@ -26,15 +26,15 @@ class TestServeCommand:
     def test_serve_with_defaults(self) -> None:
         """Test serve command with default parameters."""
         with patch("cforge.commands.server.serve.uvicorn.run") as mock_run:
-            serve()
+            invoke_typer_command(serve)
             mock_run.assert_called_once()
             args, kwargs = mock_run.call_args
             assert "mcpgateway.main:app" in args
 
     def test_serve_with_custom_host_port(self) -> None:
         """Test serve command with custom host and port."""
         with patch("cforge.commands.server.serve.uvicorn.run") as mock_run:
-            serve(host="0.0.0.0", port=8080)
+            invoke_typer_command(serve, host="0.0.0.0", port=8080)
             mock_run.assert_called_once()
             _, kwargs = mock_run.call_args
             assert kwargs.get("host") == "0.0.0.0"
@@ -43,11 +43,37 @@ def test_serve_with_custom_host_port(self) -> None:
     def test_serve_with_reload(self) -> None:
         """Test serve command with reload enabled."""
         with patch("cforge.commands.server.serve.uvicorn.run") as mock_run:
-            serve(reload=True)
+            invoke_typer_command(serve, reload=True)
             mock_run.assert_called_once()
             _, kwargs = mock_run.call_args
             assert kwargs.get("reload") is True
 
+    def test_serve_with_no_auth_sets_auth_required_false(self) -> None:
+        """Test serve command with --no-auth sets auth_required to False."""
+        with patch("cforge.commands.server.serve.uvicorn.run") as mock_run, patch("cforge.commands.server.serve.set_serve_settings") as mock_set_settings:
+            invoke_typer_command(serve, no_auth=True, headless=False)
+            mock_run.assert_called_once()
+            # When headless=False (default), UI and admin API are enabled (not headless = True)
+            # When no_auth=True, auth_required should be False (not no_auth = False)
+            mock_set_settings.assert_called_once_with(
+                mcpgateway_ui_enabled=True,
+                mcpgateway_admin_api_enabled=True,
+                auth_required=False,
+            )
+
+    def test_serve_without_no_auth_sets_auth_required_true(self) -> None:
+        """Test serve command without --no-auth sets auth_required to True."""
+        with patch("cforge.commands.server.serve.uvicorn.run") as mock_run, patch("cforge.commands.server.serve.set_serve_settings") as mock_set_settings:
+            invoke_typer_command(serve, no_auth=False, headless=False)
+            mock_run.assert_called_once()
+            # When headless=False (default), UI and admin API are enabled (not headless = True)
+            # When no_auth=False (default), auth_required should be True (not no_auth = True)
+            mock_set_settings.assert_called_once_with(
+                mcpgateway_ui_enabled=True,
+                mcpgateway_admin_api_enabled=True,
+                auth_required=True,
+            )
+
 
 class TestServeCommandIntegration:
     """Integration tests for the serve command"""
@@ -64,7 +90,7 @@ def test_serve_starts_and_responds(self, mock_settings):
         # thread does not block process exit.
         server_thread = threading.Thread(
             target=serve,
-            kwargs={"host": "127.0.0.1", "port": port, "reload": False, "workers": 1, "log_level": "error"},
+            kwargs={"host": "127.0.0.1", "port": port, "reload": False, "workers": 1, "log_level": "error", "no_auth": False},
             daemon=True,
         )
         server_thread.start()