Skip to content
This repository has been archived by the owner on Jul 17, 2023. It is now read-only.

uncache password after upload post #41

Closed
wants to merge 1 commit into from
Closed

uncache password after upload post #41

wants to merge 1 commit into from

Conversation

Curycu
Copy link

@Curycu Curycu commented Oct 22, 2019

after wiki post, Sys.getenv("CONFLUENCE_PASSWORD") shows password as plain text : so uncache password after post upload

after wiki post, Sys.getenv("CONFLUENCE_PASSWORD") shows password as plain text : so uncache password after post upload
@CLAassistant
Copy link

CLAassistant commented Oct 22, 2019

CLA assistant check
All committers have signed the CLA.

@yutannihilation
Copy link
Contributor

Sys.getenv("CONFLUENCE_PASSWORD") shows password as plain text

Could you explain when this can be an actual problem?

@Curycu
Copy link
Author

Curycu commented Oct 22, 2019

with pop up box writing : user id, pw
and then I was surprised by plain text cached my wiki password is displayed with Sys.getenv() command. I think other people also be surprised too. so add this line.

@yutannihilation
Copy link
Contributor

I know using an environmental variable is not a super cool idea, but it's a common practice to use it to store a secret because it's a (relatively) safe place; no one can look into your environment except when they has root privilege.

c.f. https://cran.r-project.org/web/packages/httr/vignettes/secrets.html

I cannot accept this PR because this would make conflr inconvenient just for protecting you from being surprised, not from real threats. Sorry.

That said, I understand you might be surprised. I'll consider adding some notice and an option not to cache this... Anyway, thanks for letting us know the issue!

@yutannihilation
Copy link
Contributor

yutannihilation commented Oct 23, 2019

@Curycu
conflr now has an option not to cache the password and username as envvars. Could you confirm adding the below into your ~/.Rprofile works when you have time?

options(conflr_dont_cache_envvar = TRUE)

@Curycu
Copy link
Author

Curycu commented Oct 30, 2019

@yutannihilation
post upload conflr asks id & pw 3 times
when I try to fix this I also had same issue. my final choice was "cache as usual and delete".

@yutannihilation
Copy link
Contributor

Ah, I see. Thanks, for the notice, I'll reopen the issue...

@yutannihilation
Copy link
Contributor

@Curycu I ended up employing your approach with an option (#48). I still doesn't feel this is the best way to do, but I haven't come up with some nicer idea than yours. Anyway, thanks for your contribution :)

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

Successfully merging this pull request may close these issues.

None yet

3 participants