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

[PR #6185/512bf4b7 backport][stable-6] ldap moduls: add optional ca_cert_file option #6217

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Mar 22, 2023

This is a backport of PR #6185 as merged into main (512bf4b).

SUMMARY

This PR adds a ca_cert_file option to the ldap modules. This option can be used to specify a ca certificate used to verify ldap tls connections.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

ldap modules

ADDITIONAL INFORMATION

I wasn't sure how to name the changelog fragment, since this PR does not address an existing issue

* add ca_cert_file option

* fix pr url

* Apply suggestions from code review

Co-authored-by: Felix Fontein <felix@fontein.de>

* update parameter name

---------

Co-authored-by: Felix Fontein <felix@fontein.de>
(cherry picked from commit 512bf4b)
@ansibullbot ansibullbot added backport docs_fragments docs_fragments plugin (shared docs) feature This issue/PR relates to a feature request module_utils module_utils new_contributor Help guide this first time contributor plugins plugin (any type) labels Mar 22, 2023
@felixfontein felixfontein merged commit 31a1c8e into stable-6 Mar 22, 2023
@felixfontein felixfontein deleted the patchback/backports/stable-6/512bf4b77f86d8e3875ed958d179d38745ba00cc/pr-6185 branch March 22, 2023 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs_fragments docs_fragments plugin (shared docs) feature This issue/PR relates to a feature request module_utils module_utils new_contributor Help guide this first time contributor plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants