search

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

R_KEYRING_KEYRING should be R_KEYRING_BACKEND #93

Closed nwstephens closed 3 years ago

nwstephens commented 3 years ago

I was surprised to find an environment variable documented as R_KEYRING_KEYRING until I looked into the code and saw that it was probably an error. I think the correct environment variable is R_KEYRING_BACKEND. There are several references to both environment variables in default_backend.R. I would expect there to be only one environment variable, and I would expect it would be named R_KEYRING_BACKEND.

https://github.com/r-lib/keyring/blob/da3e1796f606d9ce3a735a40efcef88284bada30/R/default_backend.R#L56-L57

nwstephens commented 3 years ago

Also here: https://github.com/r-lib/keyring/blob/da3e1796f606d9ce3a735a40efcef88284bada30/R/default_backend.R#L57

nwstephens commented 3 years ago

R_KEYRING_KEYRING is the default keyring. R_KEYRING_BACKEND is the default backend.