monoadmin/the-other-dude
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat(vpn): update API error handling for subnet exhaustion and IP validation

Browse Source 
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
Jason Staack
2026-03-14 16:36:46 -05:00
parent 5fb6cba4de
commit b4a7494016
1 changed files with 10 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
backend/app/routers/vpn.py
View File

@@ -83,7 +83,12 @@ async def setup_vpn(
try: try:
config = await vpn_service.setup_vpn(db, tenant_id, endpoint=body.endpoint) config = await vpn_service.setup_vpn(db, tenant_id, endpoint=body.endpoint)
except ValueError as e: except ValueError as e:
raise HTTPException(status_code=status.HTTP_409_CONFLICT, detail=str(e)) msg = str(e)
if "already configured" in msg:
raise HTTPException(status_code=status.HTTP_409_CONFLICT, detail=msg)
elif "exhausted" in msg:
raise HTTPException(status_code=422, detail=msg)
raise HTTPException(status_code=status.HTTP_409_CONFLICT, detail=msg)
return VpnConfigResponse.model_validate(config) return VpnConfigResponse.model_validate(config)
@@ -164,7 +169,10 @@ async def add_peer(
try: try:
peer = await vpn_service.add_peer(db, tenant_id, body.device_id, additional_allowed_ips=body.additional_allowed_ips) peer = await vpn_service.add_peer(db, tenant_id, body.device_id, additional_allowed_ips=body.additional_allowed_ips)
except ValueError as e: except ValueError as e:
raise HTTPException(status_code=status.HTTP_409_CONFLICT, detail=str(e)) msg = str(e)
if "must not overlap" in msg:
raise HTTPException(status_code=422, detail=msg)
raise HTTPException(status_code=status.HTTP_409_CONFLICT, detail=msg)
# Enrich with device info # Enrich with device info
result = await db.execute(select(Device).where(Device.id == peer.device_id)) result = await db.execute(select(Device).where(Device.id == peer.device_id))