[PR #5942/5648e0e2 backport][stable-6] onepassword: ignore errors fro…

…m "op account get" (#6037) onepassword: ignore errors from "op account get" (#5942) * ignore errors from "op account get" * add changelog fragment * Update changelogs/fragments/5942-onepassword-ignore-errors-from-op-account-get.yml Co-authored-by: Felix Fontein <[email protected]> --------- Co-authored-by: Felix Fontein <[email protected]> (cherry picked from commit 5648e0e) Co-authored-by: Glenn Marcy <[email protected]>
ansible-collections · Feb 22, 2023 · 1fc53ee · 1fc53ee
1 parent 3ecbadf
commit 1fc53ee
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/changelogs/fragments/5942-onepassword-ignore-errors-from-op-account-get.yml b/changelogs/fragments/5942-onepassword-ignore-errors-from-op-account-get.yml
@@ -0,0 +1,2 @@
+bugfixes:
+- onepassword lookup plugin - Changed to ignore errors from "op account get" calls. Previously, errors would prevent auto-signin code from executing (https://github.com/ansible-collections/community.general/pull/5942).
diff --git a/plugins/lookup/onepassword.py b/plugins/lookup/onepassword.py
@@ -488,7 +488,7 @@ def assert_logged_in(self):
                 account = "{subdomain}.{domain}".format(subdomain=self.subdomain, domain=self.domain)
                 args.extend(["--account", account])
 
-            rc, out, err = self._run(args)
+            rc, out, err = self._run(args, ignore_errors=True)
 
             return not bool(rc)