Merge pull request #1115 from NickCrews/domain-creds

(fixed #1113) Supply correct Google API v3 permission for domains
burnash · Oct 9, 2022 · 409f96d · 409f96d
2 parents b5109bb + 2770e5e
commit 409f96d
Showing 1 changed file with 11 additions and 5 deletions.
diff --git a/gspread/client.py b/gspread/client.py
@@ -491,20 +491,26 @@ def insert_permission(
         """
 
         url = "{}/{}/permissions".format(DRIVE_FILES_API_V3_URL, file_id)
-
         payload = {
-            "emailAddress": value,
             "type": perm_type,
             "role": role,
             "withLink": with_link,
         }
-
         params = {
-            "sendNotificationEmail": notify,
-            "emailMessage": email_message,
             "supportsAllDrives": "true",
         }
 
+        if perm_type == "domain":
+            payload["domain"] = value
+        elif perm_type in {"user", "group"}:
+            payload["emailAddress"] = value
+            params["sendNotificationEmail"] = notify
+            params["emailMessage"] = email_message
+        elif perm_type == "anyone":
+            pass
+        else:
+            raise ValueError("Invalid permission type: {}".format(perm_type))
+
         return self.request("post", url, json=payload, params=params)
 
     def remove_permission(self, file_id, permission_id):