fix: Add now required content to JSONResponse

Due to fastapi/fastapi#4488
betagouv · Aug 8, 2022 · 83841e9 · 83841e9
1 parent eb00c5f
commit 83841e9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/api/deployments.py b/api/deployments.py
@@ -22,7 +22,7 @@ def get_deployment_status(
     try:
         status = azure_client.get_deployment_status(project_name)
     except DeploymentNotFound:
-        return JSONResponse(status_code=404)
+        return JSONResponse(status_code=404, content={})
     return {"status": status}
 
 

diff --git a/api/vms.py b/api/vms.py
@@ -19,7 +19,7 @@ def delete_vm(
 ):
     """Delete a VM and its connection information on Guacamole."""
     if not current_user.is_admin:
-        return JSONResponse(status_code=403)
+        return JSONResponse(status_code=403, content={})
     azure_client.delete_vm(project_name)
     azure_client.delete_deployment(
         project_name