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.

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

Setting the keyring password with the file backend #106

Closed
nwstephens opened this issue May 6, 2021 · 3 comments
Closed

Setting the keyring password with the file backend #106

nwstephens opened this issue May 6, 2021 · 3 comments
Labels
bug an unexpected problem or unintended behavior

Comments

@nwstephens
Copy link

I am using the file backend with RStudio Server. When set a key, and no keyring exists, I get prompted to set a keyring password. If I hit cancel, I get a second prompt to enter a password (presumably to unlock the keyring). I would not expect to get this second prompt. Instead, I would expect cancel to mean cancel, and return me to the console. You can see the behavior here:

https://vimeo.com/545784420

@nwstephens
Copy link
Author

@gaborcsardi This issue came about when I was testing askForSecret() with the pro drivers. Calling keyring_create() from askForSecret() makes for a complicated workflow. It might be worth removing the creation of the keyring from the setting of the keyring. In other words, maybe users should have to explicitly create a keyring using the keyring_create command.

@gaborcsardi gaborcsardi added the bug an unexpected problem or unintended behavior label May 6, 2021
@gaborcsardi
Copy link
Member

Need to check for cancelling, i.e. whether askpass::askpass() returns NULL.

@nwstephens
Copy link
Author

Yes, it does. When I click NULL here is what happens in the console:

> askpass::askpass()
NULL

gaborcsardi added a commit that referenced this issue Nov 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug an unexpected problem or unintended behavior
Projects
None yet
Development

No branches or pull requests

2 participants