-
Notifications
You must be signed in to change notification settings - Fork 245
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
TDE: specify AES implementation is done with OpenSSL #6030
Conversation
768b0db
to
868287e
Compare
product_docs/docs/tde/15/index.mdx
Outdated
### How does TDE encrypt data? | ||
### How does TDE encrypt data? | ||
|
||
EDB TDE uses [OpenSSL](https://openssl-library.org/) to encrypt data files with the AES encryption algorithm. In Windows systems, TDE uses [OpenSSL 3](https://docs.openssl.org/3.0/). In Linux systems, TDE uses the OpenSSL version installed in the host operating system. To check the installed version, run `openssl version`. For more details, refer to the [OpenSSL documentation](https://docs.openssl.org/master/). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I used the wording "OpenSSL that their operating system installs as a server package dependency" just because some customers will use a build of OpenSSL that does not come from the OpenSSL community e.g.. With the wording change mentioned here, I'd change the last sentence to "For more information, see the OpenSSL documentation. If you're using a custom build not provided by the OpenSSL community, consult your vendor's documentation."
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense. I made the changes ✅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
868287e
to
95bd99c
Compare
a55215d
to
7bd2713
Compare
What Changed?
https://enterprisedb.atlassian.net/browse/DOCS-1012