youtrack-cli 0.3.3__tar.gz → 0.3.4__tar.gz

{youtrack_cli-0.3.3 → youtrack_cli-0.3.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: youtrack-cli
-Version: 0.3.3
+Version: 0.3.4
 Summary: YouTrack CLI - Command line interface for JetBrains YouTrack issue tracking system
 Project-URL: Homepage, https://github.com/ryan-murphy/yt-cli
 Project-URL: Documentation, https://yt-cli.readthedocs.io/

{youtrack_cli-0.3.3 → youtrack_cli-0.3.4}/docs/commands/admin.rst

@@ -552,6 +552,15 @@ Common Issues and Solutions
 **Health Check Issues**
   Review system logs and resource availability.
 
+**Health Check 404 Errors**
+  If the health check command returns a 404 error, this may indicate:
+
+  * Your YouTrack version doesn't support the system settings endpoint
+  * The API endpoint has changed in your YouTrack version
+  * Your YouTrack instance has a different API configuration
+
+  The command will automatically try fallback endpoints and provide specific guidance based on the error type.
+
 System Diagnostics
 ~~~~~~~~~~~~~~~~~
 

{youtrack_cli-0.3.3 → youtrack_cli-0.3.4}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "youtrack-cli"
-version = "0.3.3"
+version = "0.3.4"
 description = "YouTrack CLI - Command line interface for JetBrains YouTrack issue tracking system"
 readme = "README.md"
 requires-python = ">=3.9, <3.14"

{youtrack_cli-0.3.3 → youtrack_cli-0.3.4}/tests/test_admin.py

@@ -175,6 +175,38 @@ class TestAdminManager:
             assert result["status"] == "success"
             assert result["data"] == mock_health
 
+    @pytest.mark.asyncio
+    async def test_get_system_health_404_error(self, admin_manager, auth_manager):
+        """Test system health check with 404 error on all endpoints."""
+        with patch("httpx.AsyncClient") as mock_client:
+            mock_response = Mock()
+            mock_response.status_code = 404
+            mock_request = Mock()
+            http_error = httpx.HTTPStatusError("Not Found", request=mock_request, response=mock_response)
+            mock_client.return_value.__aenter__.return_value.get.side_effect = http_error
+
+            result = await admin_manager.get_system_health()
+
+            assert result["status"] == "error"
+            assert "System health endpoint not found (404)" in result["message"]
+            assert "YouTrack version doesn't support this endpoint" in result["message"]
+
+    @pytest.mark.asyncio
+    async def test_get_system_health_403_error(self, admin_manager, auth_manager):
+        """Test system health check with 403 permission error."""
+        with patch("httpx.AsyncClient") as mock_client:
+            mock_response = Mock()
+            mock_response.status_code = 403
+            mock_request = Mock()
+            http_error = httpx.HTTPStatusError("Forbidden", request=mock_request, response=mock_response)
+            mock_client.return_value.__aenter__.return_value.get.side_effect = http_error
+
+            result = await admin_manager.get_system_health()
+
+            assert result["status"] == "error"
+            assert "Insufficient permissions for health check" in result["message"]
+            assert "Low-level Admin Read" in result["message"]
+
     @pytest.mark.asyncio
     async def test_clear_caches_success(self, admin_manager, auth_manager):
         """Test successful cache clearing."""

{youtrack_cli-0.3.3 → youtrack_cli-0.3.4}/uv.lock

@@ -1751,7 +1751,7 @@ wheels = [
 
 [[package]]
 name = "youtrack-cli"
-version = "0.3.3"
+version = "0.3.4"
 source = { editable = "." }
 dependencies = [
     { name = "click" },

{youtrack_cli-0.3.3 → youtrack_cli-0.3.4}/youtrack_cli/admin.py

@@ -153,6 +153,7 @@ class AdminManager:
                 response = await client.get(
                     f"{credentials.base_url.rstrip('/')}/api/admin/globalSettings/license",
                     headers=headers,
+                    params={"fields": "id,username,license,error"},
                     timeout=10.0,
                 )
                 response.raise_for_status()
@@ -167,6 +168,14 @@ class AdminManager:
                             "status": "error",
                             "message": "Insufficient permissions to view license.",
                         }
+                    elif e.response.status_code == 404:
+                        return {
+                            "status": "error",
+                            "message": (
+                                "License endpoint not found. This may indicate an "
+                                "incompatible YouTrack version or configuration."
+                            ),
+                        }
                 return {"status": "error", "message": f"HTTP error: {e}"}
             except Exception as e:
                 return {"status": "error", "message": f"Unexpected error: {e}"}
@@ -231,28 +240,62 @@ class AdminManager:
             "Accept": "application/json",
         }
 
-        async with httpx.AsyncClient() as client:
-            try:
-                response = await client.get(
-                    f"{credentials.base_url.rstrip('/')}/api/admin/globalSettings/systemSettings",
-                    headers=headers,
-                    timeout=10.0,
-                )
-                response.raise_for_status()
-
-                health_info = response.json()
-                return {"status": "success", "data": health_info}
+        # List of endpoints to try, in order of preference
+        endpoints = [
+            "/api/admin/globalSettings/systemSettings",
+            "/api/admin/globalSettings/systemSettings?fields=baseUrl,isApplicationReadOnly,maxUploadFileSize,maxExportItems",
+        ]
 
-            except httpx.HTTPError as e:
-                if hasattr(e, "response") and e.response is not None:
-                    if e.response.status_code == 403:
+        async with httpx.AsyncClient() as client:
+            for endpoint in endpoints:
+                try:
+                    response = await client.get(
+                        f"{credentials.base_url.rstrip('/')}{endpoint}",
+                        headers=headers,
+                        timeout=10.0,
+                    )
+                    response.raise_for_status()
+
+                    health_info = response.json()
+                    return {"status": "success", "data": health_info}
+
+                except httpx.HTTPStatusError as e:
+                    if e.response.status_code == 404:
+                        continue  # Try next endpoint
+                    elif e.response.status_code == 403:
                         return {
                             "status": "error",
-                            "message": "Insufficient permissions for health check.",
+                            "message": "Insufficient permissions for health check. "
+                            "Requires 'Low-level Admin Read' permission.",
                         }
-                return {"status": "error", "message": f"HTTP error: {e}"}
-            except Exception as e:
-                return {"status": "error", "message": f"Unexpected error: {e}"}
+                    elif e.response.status_code == 401:
+                        return {
+                            "status": "error",
+                            "message": "Authentication failed. Your token may have expired. Run 'yt auth login' again.",
+                        }
+                    else:
+                        response_text = e.response.text if hasattr(e.response, "text") else str(e)
+                        return {
+                            "status": "error",
+                            "message": f"HTTP {e.response.status_code}: {response_text}",
+                        }
+                except httpx.RequestError as e:
+                    return {
+                        "status": "error",
+                        "message": f"Network error: {e}. Check your YouTrack URL and network connection.",
+                    }
+                except Exception as e:
+                    return {"status": "error", "message": f"Unexpected error: {e}"}
+
+            # If all endpoints failed with 404
+            return {
+                "status": "error",
+                "message": "System health endpoint not found (404). This may indicate:\n"
+                "1. Your YouTrack version doesn't support this endpoint\n"
+                "2. The endpoint URL may have changed\n"
+                "3. Your YouTrack instance has a different API configuration\n"
+                "Please verify your YouTrack version and API access.",
+            }
 
     async def clear_caches(self) -> dict[str, Any]:
         """Clear system caches.