-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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: Updated LDAP Documentation #1988
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1988 +/- ##
===========================================
- Coverage 78.10% 44.87% -33.24%
===========================================
Files 71 71
Lines 8611 8479 -132
===========================================
- Hits 6726 3805 -2921
- Misses 1885 4674 +2789
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
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.
Thank you!
left just a really small comment
docs/config.rst
Outdated
+----------------------------------------+--------------------------------------------+-----------+ | ||
| AUTH_LDAP_TLS_CACERTFILE | CA Certificate file to check peer | No | | ||
| | certificate | | | ||
| | certificate. FIle must be PEM-encoded | | |
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.
File must be PEM-encoded
(change the upper I)
Description
After spending some time trying to set up Apache Airflow with LDAP-over-TLS authentication and running into multiple TLS issues, I added some missing documentation that would helped me.
ADDITIONAL INFORMATION