Skip to content

Conversation

@sleto-it
Copy link
Contributor

No description provided.

@sleto-it sleto-it requested a review from ewoutp June 25, 2018 11:22
Copy link
Contributor

@ewoutp ewoutp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Few changes needed

============================

The password that is set for the _root_ user during the installation of the ArangoDB
package has no effect in case of deployments done with the tool _ArangoDB Stater_,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stater -> Starter_

as this tool creates new database directories and configuration files that are
separate from those created by the stand-alone installation.

Assuming you have enabled authentication in your _Starter_ deployment, by default
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

... using --auth.jwt-secret=<thesecret-file> ...


where "mypwd" is the new password you want to set.

If your _Starter_ deployment has authentication turned off, it is suggested to
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should add a note saying that you cannot change authentication from on/off after you created the deployment once.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the feedback. If the user shutdown their cluster, and manually change all the configuration files (for a 3 nodes cluster 9 files in total), and in all the config filed change authentication = true (from false) and add jwt-secret = the-secret-here and then restart the cluster (even using the old starter commands that do not include --auth.jwt-secret= it should now work with authentication on, right? I think we have tested this procedure before. I am a bit unsure right now if the .txt command files have to be changed as well, or not. Otherwise we can still suggest to create a new fresh cluster, with auth enabled (and in case restore your data there). WDYT?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should work. However I prefer to keep this as a "don't do it like this" and implement changing the config files properly.

@ewoutp
Copy link
Contributor

ewoutp commented Jun 26, 2018

@sleto-it I've made a couple of modifications. Please see if you agree.

@ewoutp
Copy link
Contributor

ewoutp commented Jun 28, 2018

discussed with @sleto-it merge now and go over it once more in the grant docs overhaul.

@ewoutp ewoutp merged commit 17711d8 into master Jun 28, 2018
@ewoutp ewoutp deleted the documentation/security branch June 28, 2018 10:49
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 this pull request may close these issues.

3 participants