Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Cannot obtain secret fields with op 2.30.0 or later #64

Open
csteinle opened this issue Sep 6, 2024 · 2 comments · May be fixed by #65
Open

Cannot obtain secret fields with op 2.30.0 or later #64

csteinle opened this issue Sep 6, 2024 · 2 comments · May be fixed by #65

Comments

@csteinle
Copy link

csteinle commented Sep 6, 2024

Describe the bug
Since 2.30.0 of the 1password CLI, the option --reveal is required to obtain passwords and other secret fields. See https://app-updates.agilebits.com/product_history/CLI2#v2300006

To Reproduce

from onepassword import OnePassword

op = OnePassword()

uuid = op.get_uuid("Item Name", vault="Vault Name")

token = op.get_item(uuid=uuid, fields="token")["token"]

print(token)
[use 'op item get <redacted> --reveal' to reveal]

Expected behavior
Password or token retrieved

Screenshots
N/A

Desktop (please complete the following information):
op CLI version 2.30.0 or greater

Smartphone (please complete the following information):
N/A

Additional context
N/A
@dtpryce
Copy link
Collaborator

dtpryce commented Sep 6, 2024

We should update the README to mention that we try to support a fixed version of the cli, but yeah this looks like a backward incompatiblity. Any chance you can create a PR for this otherwise we will try to fix as soon as we can (might be some time) or just suggest people to downgrade op cli for now.

@csteinle csteinle linked a pull request Sep 6, 2024 that will close this issue
Open
@csteinle
Copy link
Author

csteinle commented Sep 6, 2024

I've submitted a PR (with some unit tests specifically for this change)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

add --reveal to cli for v2.30.0 and later csteinle/1password-client
2 participants
@dtpryce @csteinle