From ee965a1a4c1e122c5b0adb93c548b41d587bd0a2 Mon Sep 17 00:00:00 2001
From: Ruben Fiszel <ruben@rubenfiszel.com>
Date: Fri, 26 Jan 2024 09:33:22 +0100
Subject: [PATCH] fix: git sync include changing permissions on apps, scripts,
 flows

---
 backend/windmill-api/src/granular_acls.rs | 87 +++++++++++++++++++++++
 1 file changed, 87 insertions(+)

diff --git a/backend/windmill-api/src/granular_acls.rs b/backend/windmill-api/src/granular_acls.rs
index 8f2ba64627..23f74c73ea 100644
--- a/backend/windmill-api/src/granular_acls.rs
+++ b/backend/windmill-api/src/granular_acls.rs
@@ -20,6 +20,7 @@ use serde::{Deserialize, Serialize};
 use windmill_common::{
     db::UserDB,
     error::{Error, JsonResult, Result},
+    scripts::ScriptHash,
     utils::{not_found_if_none, StripPath},
 };
 
@@ -95,6 +96,49 @@ async fn add_granular_acl(
             )
             .await?
         }
+        "app" => {
+            handle_deployment_metadata(
+                &authed.email,
+                &authed.username,
+                &db,
+                &w_id,
+                DeployedObject::App { path: path.to_string(), parent_path: None, version: 0 },
+                Some(format!("App '{}' changed permissions", path)),
+                rsmq,
+                true,
+            )
+            .await?
+        }
+        "script" => {
+            handle_deployment_metadata(
+                &authed.email,
+                &authed.username,
+                &db,
+                &w_id,
+                DeployedObject::Script {
+                    path: path.to_string(),
+                    parent_path: None,
+                    hash: ScriptHash(0),
+                },
+                Some(format!("Script '{}' changed permissions", path)),
+                rsmq,
+                true,
+            )
+            .await?
+        }
+        "flow" => {
+            handle_deployment_metadata(
+                &authed.email,
+                &authed.username,
+                &db,
+                &w_id,
+                DeployedObject::Flow { path: path.to_string(), parent_path: None },
+                Some(format!("Flow '{}' changed permissions", path)),
+                rsmq,
+                true,
+            )
+            .await?
+        }
         _ => (),
     }
 
@@ -165,6 +209,49 @@ async fn remove_granular_acl(
             )
             .await?
         }
+        "app" => {
+            handle_deployment_metadata(
+                &authed.email,
+                &authed.username,
+                &db,
+                &w_id,
+                DeployedObject::App { path: path.to_string(), parent_path: None, version: 0 },
+                Some(format!("App '{}' changed permissions", path)),
+                rsmq,
+                true,
+            )
+            .await?
+        }
+        "script" => {
+            handle_deployment_metadata(
+                &authed.email,
+                &authed.username,
+                &db,
+                &w_id,
+                DeployedObject::Script {
+                    path: path.to_string(),
+                    parent_path: None,
+                    hash: ScriptHash(0),
+                },
+                Some(format!("Script '{}' changed permissions", path)),
+                rsmq,
+                true,
+            )
+            .await?
+        }
+        "flow" => {
+            handle_deployment_metadata(
+                &authed.email,
+                &authed.username,
+                &db,
+                &w_id,
+                DeployedObject::Flow { path: path.to_string(), parent_path: None },
+                Some(format!("Flow '{}' changed permissions", path)),
+                rsmq,
+                true,
+            )
+            .await?
+        }
         _ => (),
     }