Consider adding support for 'key_list' for 'env' backend

r-lib / keyring

:closed_lock_with_key: Access the system credential store from R

https://keyring.r-lib.org/

Other

196 stars 28 forks source link

Consider adding support for 'key_list' for 'env' backend #57

Closed javierluraschi closed 3 years ago

javierluraschi commented 6 years ago

It would be great to support 'key_list' in 'keyring', at least for named services since this is a common configuration in RStudio Server while using .rs.askForSecret() in the RStudio API, will send a PR.

gaborcsardi commented 6 years ago

The env backend already has a list method, but it only works if the service is given. Isn't that enough?

gaborcsardi commented 3 years ago

With making the 'file' backend the default fallback now, I don't think this is very important. Especially that it is actually hard/impossible to implement without changing either the env var names or their contents. Because currently any env var can be a secret as far as keyring is concerned.

@javierluraschi Please let know if you feel otherwise.