diff --git a/api_server/routes/internal/internal_routes.py b/api_server/routes/internal/internal_routes.py index 1477afa01..7f7a840de 100644 --- a/api_server/routes/internal/internal_routes.py +++ b/api_server/routes/internal/internal_routes.py @@ -3,6 +3,7 @@ from typing import Optional from folder_paths import folder_names_and_paths, get_directory_by_type from api_server.services.terminal_service import TerminalService import app.logger +import ipaddress import os class InternalRoutes: @@ -72,7 +73,19 @@ class InternalRoutes: def get_app(self): if self._app is None: - self._app = web.Application() + self._app = web.Application(middlewares=[self._local_only_middleware]) self.setup_routes() self._app.add_routes(self.routes) return self._app + + @web.middleware + async def _local_only_middleware(self, request, handler): + remote = request.remote + if remote is None: + raise web.HTTPForbidden(reason="Internal endpoints are only accessible from localhost") + try: + if not ipaddress.ip_address(remote).is_loopback: + raise web.HTTPForbidden(reason="Internal endpoints are only accessible from localhost") + except ValueError: + raise web.HTTPForbidden(reason="Internal endpoints are only accessible from localhost") + return await handler(request)