Remove git version from mcpgateway.version and web ui (#418)

Signed-off-by: Mihai Criveti <[email protected]>

Author: crivetimihai

mcpgateway/templates/version_info_partial.html

@@ -30,21 +30,11 @@ <h2 class="text-2xl font-bold mb-4 flex items-center">
     </div>
   </div>
 
-  {% if payload.app.version or payload.app.git_revision %}
+  {% if payload.app.version %}
   <div class="mt-4 pt-4 border-t border-blue-300">
-    <div class="grid grid-cols-1 md:grid-cols-2 gap-4">
-      {% if payload.app.version %}
-      <div class="text-center">
-        <div class="text-lg font-medium">Version</div>
-        <div class="text-blue-100 font-mono text-sm">{{ payload.app.version }}</div>
-      </div>
-      {% endif %}
-      {% if payload.app.git_revision %}
-      <div class="text-center">
-        <div class="text-lg font-medium">Git Revision</div>
-        <div class="text-blue-100 font-mono text-sm">{{ payload.app.git_revision }}</div>
-      </div>
-      {% endif %}
+    <div class="text-center">
+      <div class="text-lg font-medium">Version</div>
+      <div class="text-blue-100 font-mono text-sm">{{ payload.app.version }}</div>
     </div>
   </div>
   {% endif %}

mcpgateway/version.py

@@ -13,7 +13,8 @@
 - Cross-platform system metrics (Windows/macOS/Linux), with fallbacks where APIs are unavailable
 - Optional dependencies: psutil (for richer metrics) and redis.asyncio (for Redis health); omitted gracefully if absent
 - Authentication enforcement via `require_auth`; unauthenticated browsers see login form, API clients get JSON 401
-- Redacted environment variables, sanitized DB/Redis URLs, Git commit detection
+- Redacted environment variables, sanitized DB/Redis URLs
+
 """
 
 # Future
@@ -25,7 +26,6 @@
 import os
 import platform
 import socket
-import subprocess
 import time
 from typing import Any, Dict, Optional
 from urllib.parse import urlsplit, urlunsplit
@@ -89,26 +89,6 @@ def _public_env() -> Dict[str, str]:
     return {k: v for k, v in os.environ.items() if not _is_secret(k)}
 
 
-def _git_revision() -> Optional[str]:
-    """
-    Retrieve the current Git revision (short) if available.
-
-    Returns:
-        Optional[str]: The Git commit hash prefix or None if unavailable.
-    """
-    rev = os.getenv("GIT_COMMIT")
-    if rev:
-        return rev[:9]
-    try:
-        out = subprocess.check_output(
-            ["git", "rev-parse", "--short", "HEAD"],
-            stderr=subprocess.DEVNULL,
-        )
-        return out.decode().strip()
-    except Exception:
-        return None
-
-
 def _sanitize_url(url: Optional[str]) -> Optional[str]:
     """
     Redact credentials from a URL for safe display.
@@ -261,7 +241,6 @@ def _build_payload(
             "name": settings.app_name,
             "version": __version__,
             "mcp_protocol_version": settings.protocol_version,
-            "git_revision": _git_revision(),
         },
         "platform": {
             "python": platform.python_version(),

tests/unit/mcpgateway/test_version.py

@@ -82,7 +82,6 @@ def _build_app(monkeypatch: pytest.MonkeyPatch, auth_ok: bool = True) -> FastAPI
     # Stub heavy helpers
     monkeypatch.setattr(ver_mod, "_database_version", lambda: ("db-vX", True))
     monkeypatch.setattr(ver_mod, "_system_metrics", lambda: {"stub": True})
-    monkeypatch.setattr(ver_mod, "_git_revision", lambda: "deadbeef")
     monkeypatch.setattr(ver_mod, "REDIS_AVAILABLE", False, raising=False)
 
     # Auth override
@@ -114,7 +113,6 @@ def test_version_json_ok(client: TestClient) -> None:
     payload: Dict[str, Any] = rsp.json()
     assert payload["database"]["server_version"] == "db-vX"
     assert payload["system"] == {"stub": True}
-    assert payload["app"]["git_revision"] == "deadbeef"
 
 
 def test_version_html_query_param(client: TestClient) -> None:
@@ -167,30 +165,6 @@ def test_sanitize_url() -> None:
     assert ver_mod._sanitize_url(url) == "postgres://u@host:5432/db"
 
 
-def test_git_revision_env(monkeypatch: pytest.MonkeyPatch) -> None:
-    # First-Party
-    from mcpgateway import version as ver_mod
-
-    monkeypatch.setenv("GIT_COMMIT", "0123456789abcdef")
-    assert ver_mod._git_revision() == "012345678"
-
-
-def test_git_revision_subprocess(monkeypatch: pytest.MonkeyPatch) -> None:
-    """Ensure subprocess branch returns short hash."""
-    # First-Party
-    from mcpgateway import version as ver_mod
-
-    # Remove env override
-    monkeypatch.delenv("GIT_COMMIT", raising=False)
-
-    fake_subprocess = types.SimpleNamespace(
-        check_output=lambda *a, **kw: b"abc123\n",
-        DEVNULL=-3,  # needed by _git_revision
-    )
-    monkeypatch.setattr(ver_mod, "subprocess", fake_subprocess)
-    assert ver_mod._git_revision() == "abc123"
-
-
 # --------------------------------------------------------------------------- #
 # _database_version branches                                                  #
 # --------------------------------------------------------------------------- #