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

default password broken in docklet_rstudio #170

Closed
nielsaka opened this issue Nov 10, 2018 · 2 comments · Fixed by #171
Closed

default password broken in docklet_rstudio #170

nielsaka opened this issue Nov 10, 2018 · 2 comments · Fixed by #171
Milestone

Comments

@nielsaka
Copy link
Contributor

Running the default example from the repository's README.md

docklet_create() %>%
  docklet_rstudio()

did not work out of the box. The browser opened but no connection was made. Apparently the containerized RStudio server rejects any passwords that are set to rstudio.

I found this out by loging into the droplet manually and starting the docker container, which gave

   ERROR: You must set a unique PASSWORD (not 'rstudio') first! e.g. run with:
docker run -e PASSWORD=<YOUR_PASS> -p 8787:8787 rocker/rstudio

Indeed, with a different password

docklet_create() %>%
  docklet_rstudio(password = "really_nice_r_package!")

it all works. I suggest modifying the default password here.

@nielsaka
Copy link
Contributor Author

I would like to add that the broken password is actually desired behaviour from over here at rocker. They disliked the feature of a default password that too many people end up not changing.

Not sure if setting a new default password is a good response to their change.

@sckott
Copy link
Collaborator

sckott commented Nov 12, 2018

thanks @nielsaka for raising this issue. having a look at the PR

@sckott sckott added this to the v0.7 milestone Dec 19, 2018
sckott added a commit that referenced this issue Dec 19, 2018
make both fxns have user and password as rquired params
and both check that password is not rstudio exacxtly
bump version
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

Successfully merging a pull request may close this issue.

2 participants