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

spin off new package for token management #371

Open
wibeasley opened this issue Nov 15, 2021 · 2 comments
Open

spin off new package for token management #371

wibeasley opened this issue Nov 15, 2021 · 2 comments
Assignees

Comments

@wibeasley
Copy link
Member

@RaymondBalise, @nutterb & others, depending on the recommendations of the token committee, it may make sense to pull out the security functions into a separate package with few dependencies. tidyREDCap, redcapAPI, and REDCapR could use this package as a dependency.

I'm thinking mostly about functions related to token retrieval (eg, REDCapR's existing functions) and anything else approved by the committee.

@wibeasley wibeasley self-assigned this Nov 15, 2021
@eveyp
Copy link

eveyp commented Feb 17, 2022

I'm curious if you'd considered using the keyring package for local token storage. It's an interface to the OS's built-in credential store.

It looks like a relatively simple way to boost the security of local storage for users who can't set up the DB option, but I was wondering if you'd thought of it and ruled it out already.

Thanks, and, btw really appreciate the package!!

@llrs
Copy link

llrs commented Feb 15, 2023

Hi, any update on this?

Just my 2cents: I don't think there is a need for a new package.
In other packages to handle connection to an API the required data is stored in an internal environment of the package for its usage during an R session.
So the user only needs to read them once for a given session.
To create it, credentials might be provided via keyring, askpass or other similar methods.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants