Skip to content

Added example about security enforcement#205

Open
simonzhekoff wants to merge 1 commit into
mainfrom
GDB-14611
Open

Added example about security enforcement#205
simonzhekoff wants to merge 1 commit into
mainfrom
GDB-14611

Conversation

@simonzhekoff
Copy link
Copy Markdown

Added example about security enforcement

Comment thread examples/security-enforce/README.md Outdated
initialPassword: "{bcrypt}$2a$12$VDd8PrAndaJfoMJFlHFot.osSxZWQjMQZKgrEJgVZKFj6WFPvkbnS" # admin123
provisioner:
username: provisioner
password: iHaveSuperpowers
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Or provisioner.passwordHash

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Fixed.

Comment thread examples/security-enforce/README.md Outdated
least one node has `graphdb.auth.security.enabled` set and at least one
other does not.

Apply the same `values.yaml` to every node in the cluster to avoid these
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This note is not really needed as this is always the case, unless for some reason... you do separate the helm install

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Fixed.

initialPassword: "{bcrypt}$2a$12$VDd8PrAndaJfoMJFlHFot.osSxZWQjMQZKgrEJgVZKFj6WFPvkbnS" # admin123
provisioner:
username: provisioner
passwordHash: "{bcrypt}$2b$12$kCFzg6FJGFRulAxbkR6VPO1URlDiiY3hE4X6NGv/4FCipHg2hxzfG" # iHaveSuperpowers
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

For some reason, the provisioner adds the bcrypt suffix automatically, see https://github.com/Ontotext-AD/graphdb-helm/blob/main/files/config/users.js#L19

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.

2 participants