Don't bother to authenticate a version check. Makes updating installs easier

Author: d-j-hatton

src/murfey/server/api.py

@@ -7,7 +7,6 @@
 from pathlib import Path
 from typing import Dict, List
 
-import packaging.version
 import sqlalchemy
 from fastapi import APIRouter, Depends, Request
 from fastapi.responses import FileResponse, HTMLResponse
@@ -1132,23 +1131,6 @@ async def request_tomography_preprocessing(
     return proc_file
 
 
-@router.get("/version")
-def get_version(client_version: str = ""):
-    result = {
-        "server": murfey.__version__,
-        "oldest-supported-client": murfey.__supported_client_version__,
-    }
-
-    if client_version:
-        client = packaging.version.parse(client_version)
-        server = packaging.version.parse(murfey.__version__)
-        minimum_version = packaging.version.parse(murfey.__supported_client_version__)
-        result["client-needs-update"] = minimum_version > client
-        result["client-needs-downgrade"] = client > server
-
-    return result
-
-
 @router.post("/visits/{visit_name}/suggested_path")
 def suggest_path(visit_name, params: SuggestedPathParameters):
     count: int | None = None

src/murfey/server/bootstrap.py

@@ -24,6 +24,7 @@
 from fastapi import APIRouter, HTTPException, Request, Response
 from fastapi.responses import FileResponse, HTMLResponse
 
+import murfey
 from murfey.server import get_machine_config, respond_with_template
 
 tag = {
@@ -38,6 +39,7 @@
 pypi = APIRouter(prefix="/pypi", tags=["bootstrap"])
 plugins = APIRouter(prefix="/plugins", tags=["bootstrap"])
 cygwin = APIRouter(prefix="/cygwin", tags=["bootstrap"])
+version = APIRouter(prefix="/version", tags=["bootstrap"])
 
 log = logging.getLogger("murfey.server.bootstrap")
 
@@ -362,3 +364,20 @@ def parse_cygwin_request(request_path: str):
         media_type=cygwin_data.headers.get("Content-Type"),
         status_code=cygwin_data.status_code,
     )
+
+
+@version.get("/")
+def get_version(client_version: str = ""):
+    result = {
+        "server": murfey.__version__,
+        "oldest-supported-client": murfey.__supported_client_version__,
+    }
+
+    if client_version:
+        client = packaging.version.parse(client_version)
+        server = packaging.version.parse(murfey.__version__)
+        minimum_version = packaging.version.parse(murfey.__supported_client_version__)
+        result["client-needs-update"] = minimum_version > client
+        result["client-needs-downgrade"] = client > server
+
+    return result

src/murfey/server/main.py

@@ -57,6 +57,7 @@ class Settings(BaseSettings):
 app.include_router(murfey.server.bootstrap.cygwin)
 app.include_router(murfey.server.bootstrap.pypi)
 app.include_router(murfey.server.bootstrap.plugins)
+app.include_router(murfey.server.bootstrap.version)
 app.include_router(murfey.server.clem.api.router)
 app.include_router(murfey.server.auth.api.router)
 app.include_router(murfey.server.websocket.ws)