updating syntax to use permissions options

Author: scottisloud

migration/cleanup-scripts/move_items_to_tracked_vault.py

@@ -37,7 +37,7 @@ def getVaults():
     vaults = []
     try:
         getVaultsCommand = subprocess.run(
-            ["op", "vault", "list", "--group=Owners", "--format=json"],
+            ["op", "vault", "list", "--permission=manage_vault", "--format=json"],
             check=True,
             capture_output=True,
         )

migration/cleanup-scripts/remove_imported_prefix.py

@@ -4,24 +4,38 @@
 
 def getVaults():
     try:
-        return subprocess.run("op vault list --group=Owners --format=json", shell=True, check=True, capture_output=True).stdout
-    except (Exception) as err:
-        print(f"Encountered an error getting the list of vaults you have access to: ", err)
+        return subprocess.run(
+            ["op", "vault", "list", "--permission=manage_vault", "--format=json"],
+            check=True,
+            capture_output=True,
+        ).stdout
+    except Exception as err:
+        print(
+            f"Encountered an error getting the list of vaults you have access to: ", err
+        )
         return
 
+
 def main():
     vaultList = json.loads(getVaults())
-    print("Removing 'Imported' prefix from all imported vaults in your 1Password account.\n\n")
+    print(
+        "Removing 'Imported' prefix from all imported vaults in your 1Password account.\n\n"
+    )
     for vault in vaultList:
         vaultID = vault["id"]
         vaultName = vault["name"]
-        if (vaultName.startswith("Imported ")):
+        if vaultName.startswith("Imported "):
             trimmedName = vaultName.removeprefix("Imported ")
             try:
-                subprocess.run(["op", "vault", "edit", vaultID, f"--name={trimmedName}"], check=True, capture_output=True)
-                print(f"\t Changed \"{vaultName}\" => \"{trimmedName}\"")
-            except (Exception) as err:
+                subprocess.run(
+                    ["op", "vault", "edit", vaultID, f"--name={trimmedName}"],
+                    check=True,
+                    capture_output=True,
+                )
+                print(f'\t Changed "{vaultName}" => "{trimmedName}"')
+            except Exception as err:
                 print(f"Encountered an error renaming {vaultName}: ", err)
                 continue
 
-main()
+
+main()

migration/cleanup-scripts/vault_dedupe_helper.py

@@ -1,5 +1,5 @@
 ###
-# A script that will produce a csv-like report to assist with de-duplication. 
+# A script that will produce a csv-like report to assist with de-duplication.
 # Outputs vaultName, vaultUUID, itemCount, vault creation date, vault updated, number of users
 ###
 import os
@@ -11,42 +11,72 @@
 scriptPath = os.path.dirname(__file__)
 outputPath = scriptPath  # Optionally choose an alternative output path here.
 
+
 # Get a list of vaults the logged-in user has access to
 def getVaults():
     try:
-        return subprocess.run("op vault list --group=Owners --format=json", shell=True, check=True, capture_output=True).stdout
-    except (Exception) as err:
-        print(f"Encountered an error getting the list of vaults you have access to: ", err)
+        return subprocess.run(
+            ["op", "vault", "list", "--permission=manage_vault", "--format=json"],
+            check=True,
+            capture_output=True,
+        ).stdout
+    except Exception as err:
+        print(
+            f"Encountered an error getting the list of vaults you have access to: ", err
+        )
         return
-    
+
+
 # Get a list of users and their permissions for a vault
 def getVaultUserList(vaultID):
     try:
-        return subprocess.run(f"op vault user list {vaultID} --format=json", shell=True, check=True, capture_output=True).stdout
-    except (Exception) as err:
-        print(f"Encountered an error getting the list of users for vault {vaultID}: ", err)
+        return subprocess.run(
+            f"op vault user list {vaultID} --format=json",
+            shell=True,
+            check=True,
+            capture_output=True,
+        ).stdout
+    except Exception as err:
+        print(
+            f"Encountered an error getting the list of users for vault {vaultID}: ", err
+        )
         return
 
+
 # Get the details of a vault
 def getVaultDetails(vaultID):
     try:
-        return subprocess.run(f"op vault get {vaultID} --format=json", shell=True, check=True, capture_output=True).stdout
-    except (Exception) as err:
+        return subprocess.run(
+            f"op vault get {vaultID} --format=json",
+            shell=True,
+            check=True,
+            capture_output=True,
+        ).stdout
+    except Exception as err:
         print(f"Encountered an error getting details for vault {vaultID}: ", err)
         return
 
+
 # Make dates returned from the CLI a bit nicer
 def formatDate(date):
     isoDate = datetime.fromisoformat(date)
     return f"{isoDate.year}-{isoDate.month}-{isoDate.day} {isoDate.hour}:{isoDate.minute}:{isoDate.second}"
 
+
 def main():
     vaultList = json.loads(getVaults())
     with open(f"{outputPath}/vaultreport.csv", "w", newline="") as outputFile:
         csvWriter = csv.writer(outputFile)
-        fields = ["vaultName", "vaultUUD", "itemCount", "vaultCreated", "vaultUpdated", "userCount"]
+        fields = [
+            "vaultName",
+            "vaultUUD",
+            "itemCount",
+            "vaultCreated",
+            "vaultUpdated",
+            "userCount",
+        ]
         csvWriter.writerow(fields)
-        
+
         for vault in vaultList:
             if vault["name"] == "Private":
                 continue
@@ -56,19 +86,25 @@ def main():
             dateCreated = formatDate(vaultDetails["created_at"])
             dateUpdated = formatDate(vaultDetails["updated_at"])
 
-            csvWriter.writerow([
+            csvWriter.writerow(
+                [
+                    vault["name"],
+                    vault["id"],
+                    vaultDetails["items"],
+                    dateCreated,
+                    dateUpdated,
+                    userCount,
+                ]
+            )
+
+            print(
                 vault["name"],
                 vault["id"],
                 vaultDetails["items"],
                 dateCreated,
                 dateUpdated,
-                userCount
-            ]) 
-            
-            print(vault["name"],
-                vault["id"],
-                vaultDetails["items"],
-                dateCreated,
-                dateUpdated,
-                userCount)
-main()
+                userCount,
+            )
+
+
+main()