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

[docs] Update security-checklist-yp.md #9071

Merged
merged 2 commits into from
Jul 14, 2021

Conversation

kerneljake
Copy link
Contributor

Add links to references.

Add links to references.
@kerneljake kerneljake added the area/documentation Documentation needed label Jun 23, 2021
@kerneljake kerneljake requested a review from polarweasel June 23, 2021 19:04
@netlify
Copy link

netlify bot commented Jun 23, 2021

✔️ Deploy Preview for infallible-bardeen-164bc9 ready!

🔨 Explore the source changes: 9d98613

🔍 Inspect the deploy log: https://app.netlify.com/sites/infallible-bardeen-164bc9/deploys/60ef52df88cba400089fd3b5

😎 Browse the preview: https://deploy-preview-9071--infallible-bardeen-164bc9.netlify.app

@kerneljake kerneljake changed the title Update security-checklist-yp.md [docs] Update security-checklist-yp.md Jun 25, 2021
@@ -23,7 +23,7 @@ Read more about enabling [Encryption in transit](../enable-encryption-at-rest) i
## Enable encryption at rest

[Encryption at rest](https://en.wikipedia.org/wiki/Data_at_rest#Encryption) ensures that data at rest, stored on disk, is protected. You can configure YugabyteDB with a user-generated symmetric key to perform cluster-wide encryption.
Read more about enabling Encryption at rest in YugabyteDB.
Read more about [enabling Encryption at rest](../enable-encryption-at-rest) in YugabyteDB.
Copy link
Contributor

Choose a reason for hiding this comment

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

This link is right, but the one on line 21 is incorrect! (That one should point to ../enable-encryption-in-transit)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think my latest commit fixes it.

encryption in transit link
Copy link
Contributor

@polarweasel polarweasel left a comment

Choose a reason for hiding this comment

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

All good!

@polarweasel polarweasel merged commit 8afa346 into yugabyte:master Jul 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Documentation needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants