vleeuwenmenno/dotfiles
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat: enhance 1Password lookup module to handle empty output and improve logging
Some checks failed
Nix Format Check / check-format (push) Failing after 38s
Details

Browse Source
This commit is contained in:
Menno van Leeuwen 2025-03-11 20:22:17 +01:00
parent 97609e5578
commit 61f29fb7ba
Signed by: vleeuwenmenno
SSH Key Fingerprint: SHA256:OJFmjANpakwD3F2Rsws4GLtbdz1TJ5tkQF0RZmF0TRE
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
config/ansible/plugins/lookup/onepassword.py
View File

@ -70,7 +70,13 @@ class LookupModule(LookupBase):
text=True, text=True,
check=True check=True
) )
return [result.stdout.strip()] output = result.stdout.strip()
display.vvv(f"1Password output for ref '{ref}': '{output}'")
if not output:
display.warning(f"1Password returned empty output for ref '{ref}'")
return [output]
except subprocess.CalledProcessError as e: except subprocess.CalledProcessError as e:
error_msg = e.stderr.strip() error_msg = e.stderr.strip()
display.warning(f"Error executing 1Password CLI: {error_msg}") display.warning(f"Error executing 1Password CLI: {error_msg}")