-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Kerberos docs #7993
Kerberos docs #7993
Conversation
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.
👍
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.
Minor stuff, LGTM!
- `starttls` `(bool: false)` – If true, issues a `StartTLS` command after | ||
establishing an unencrypted connection. | ||
- `tls_min_version` `(string: tls12)` – Minimum TLS version to use. Accepted | ||
values are `tls10`, `tls11` or `tls12`. |
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'm surprised we even allow TLS 1.0.
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.
Yeah. It's mainly to support legacy applications that don't support anything higher than tls10
. I think the idea is, something's better than nothing.
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! Minor feedback.
6894928
This PR provides documentation for the
kerberos
auth method being added in #7908. I've viewed all these docs locally and they render nicely.