Merge pull request #9332 from OpenMined/bschell/turn-off-redoc

Remove default /redoc and /docs FastAPI endpoints from server
OpenMined · Oct 1, 2024 · bd8a1c2 · bd8a1c2
2 parents 9af1e36 + 3f0b0f2
commit bd8a1c2
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 5 deletions.
diff --git a/packages/grid/backend/grid/main.py b/packages/grid/backend/grid/main.py
@@ -76,6 +76,8 @@ def create_app() -> FastAPI:
         openapi_url=f"{settings.API_V2_STR}/openapi.json",
         lifespan=lifespan,
         middleware=get_middlewares(),
+        docs_url=None,
+        redoc_url=None,
     )
 
     # instrument app

diff --git a/packages/grid/helm/syft/templates/proxy/proxy-configmap.yaml b/packages/grid/helm/syft/templates/proxy/proxy-configmap.yaml
@@ -38,7 +38,7 @@ data:
             - "web"
           service: "frontend"
         backend:
-          rule: "(PathPrefix(`/api`) || PathPrefix(`/docs`) || PathPrefix(`/redoc`)) && !PathPrefix(`/rtunnel`)"
+          rule: "PathPrefix(`/api`) && !PathPrefix(`/rtunnel`)"
           entryPoints:
             - "web"
           service: "backend"

diff --git a/packages/grid/traefik/docker/dynamic-tls.yml b/packages/grid/traefik/docker/dynamic-tls.yml
@@ -24,13 +24,13 @@ http:
         - websecure
       service: "frontend"
     backend:
-      rule: "PathPrefix(`/api`) || PathPrefix(`/docs`) || PathPrefix(`/redoc`)"
+      rule: "PathPrefix(`/api`)"
       entryPoints:
         - web
         - websecure
       service: "backend"
     backend-stream:
-      rule: "PathPrefix(`/api`) && PathPrefix(`/api/v1/syft/stream`) || PathPrefix(`/docs`) || PathPrefix(`/redoc`)"
+      rule: "PathPrefix(`/api`) && PathPrefix(`/api/v1/syft/stream`)"
       entryPoints:
         - web
         - websecure

diff --git a/packages/grid/traefik/docker/dynamic.yml b/packages/grid/traefik/docker/dynamic.yml
@@ -27,12 +27,12 @@ http:
         - web
       service: "frontend"
     backend:
-      rule: "PathPrefix(`/api`) || PathPrefix(`/docs`) || PathPrefix(`/redoc`)"
+      rule: "PathPrefix(`/api`)"
       entryPoints:
         - web
       service: "backend"
     backend-stream:
-      rule: "PathPrefix(`/api`) && PathPrefix(`/api/v1/syft/stream`) || PathPrefix(`/docs`) || PathPrefix(`/redoc`)"
+      rule: "PathPrefix(`/api`) && PathPrefix(`/api/v1/syft/stream`)"
       entryPoints:
         - web
       service: "backend-stream"