hansjone/netx
1
0
Fork 0
mirror of https://github.com/hansjone/netx.git synced 2026-10-09 09:23:17 +08:00
Code Issues Projects Releases 1 Packages Wiki Activity Actions 2

fix(managed-ne): keep API list behavior for UI while MCP remains restricted

Browse source 
Co-authored-by: Cursor <cursoragent@cursor.com>
This commit is contained in:
oliver 2026-07-02 11:24:49 +08:00
parent 987b501e53
commit 3a35b54f0e
2 changed files with 4 additions and 12 deletions
Download patch file Download diff file Expand all files Collapse all files

4
netx_api/ne_service.py
View file

@ -238,10 +238,6 @@ def list_managed_ne(
kw = str(keyword or "").strip()
v = str(vendor or "").strip()
cs = str(connect_status or "").strip()
if not (kw or v or cs):
raise HTTPException(status_code=400, detail="managed_ne_filter_required")
if kw and len(kw) < 2:
raise HTTPException(status_code=400, detail="managed_ne_keyword_too_short")
if kw:
like = f"%{kw}%"
stmt = stmt.filter(

12
tests/test_managed_ne.py
View file

@ -145,15 +145,11 @@ class ManagedNeApiTests(unittest.TestCase):
self.assertEqual(listed.json()["total"], 1, listed.text)
self.client.delete(f"/v1/managed-ne/{ne_id}")
def test_list_requires_filter(self):
def test_list_allows_empty_and_short_keyword(self):
listed = self.client.get("/v1/managed-ne")
self.assertEqual(listed.status_code, 400)
self.assertEqual(listed.json()["detail"], "managed_ne_filter_required")
def test_list_rejects_short_keyword(self):
listed = self.client.get("/v1/managed-ne", params={"keyword": "r"})
self.assertEqual(listed.status_code, 400)
self.assertEqual(listed.json()["detail"], "managed_ne_keyword_too_short")
self.assertEqual(listed.status_code, 200)
short_kw = self.client.get("/v1/managed-ne", params={"keyword": "r"})
self.assertEqual(short_kw.status_code, 200)
def test_list_allows_vendor_only_filter(self):
created = self.client.post(