From f7f17fcfd6af17f15b7605aa6f6162491a5a6481 Mon Sep 17 00:00:00 2001
From: Vitaliy Kucheryaviy <ppr.vitaly@gmail.com>
Date: Sun, 25 Aug 2019 03:39:48 +0300
Subject: [PATCH] :sparkles: Allow empty routed path (issue #414) (#415)

---
 fastapi/routing.py         |  7 ++++++-
 tests/test_empty_router.py | 33 +++++++++++++++++++++++++++++++++
 2 files changed, 39 insertions(+), 1 deletion(-)
 create mode 100644 tests/test_empty_router.py

diff --git a/fastapi/routing.py b/fastapi/routing.py
index 304b2a116..33a7d49d1 100644
--- a/fastapi/routing.py
+++ b/fastapi/routing.py
@@ -201,7 +201,6 @@ class APIRoute(routing.Route):
         response_class: Type[Response] = JSONResponse,
         dependency_overrides_provider: Any = None,
     ) -> None:
-        assert path.startswith("/"), "Routed paths must always start with '/'"
         self.path = path
         self.endpoint = endpoint
         self.name = get_name(endpoint) if name is None else name
@@ -448,6 +447,12 @@ class APIRouter(routing.Router):
             assert not prefix.endswith(
                 "/"
             ), "A path prefix must not end with '/', as the routes will start with '/'"
+        else:
+            for r in router.routes:
+                if not r.path:
+                    raise Exception(
+                        f"Prefix and path cannot be both empty (operation: {r.name})"
+                    )
         if responses is None:
             responses = {}
         for route in router.routes:
diff --git a/tests/test_empty_router.py b/tests/test_empty_router.py
new file mode 100644
index 000000000..57dd006fa
--- /dev/null
+++ b/tests/test_empty_router.py
@@ -0,0 +1,33 @@
+import pytest
+from fastapi import APIRouter, FastAPI
+from starlette.testclient import TestClient
+
+app = FastAPI()
+
+router = APIRouter()
+
+
+@router.get("")
+def get_empty():
+    return ["OK"]
+
+
+app.include_router(router, prefix="/prefix")
+
+
+client = TestClient(app)
+
+
+def test_use_empty():
+    with client:
+        response = client.get("/prefix")
+        assert response.json() == ["OK"]
+
+        response = client.get("/prefix/")
+        assert response.status_code == 404
+
+
+def test_include_empty():
+    # if both include and router.path are empty - it should raise exception
+    with pytest.raises(Exception):
+        app.include_router(router)