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 #6648/88014635 backport][stable-7] plugins/modules/ldap_search: Add support for multipage searches #6697

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jun 15, 2023

This is a backport of PR #6648 as merged into main (8801463).

SUMMARY

Add a new page_size option to ldap_search to enable paged searches.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/modules/ldap_search.py

* Add more integration tests for ldap_search

* Add new page_size option to ldap_search

* Add changelog fragment

* Apply suggestions from code review

Co-authored-by: Felix Fontein <[email protected]>

* Simplify if statement to reduce negatives

* Apply suggestions from code review

Co-authored-by: Felix Fontein <[email protected]>

---------

Co-authored-by: Felix Fontein <[email protected]>
(cherry picked from commit 8801463)
@ansibullbot
Copy link
Collaborator

@ansibullbot ansibullbot added backport feature This issue/PR relates to a feature request integration tests/integration module module new_contributor Help guide this first time contributor plugins plugin (any type) tests tests labels Jun 15, 2023
@felixfontein felixfontein merged commit 7dcbb1a into stable-7 Jun 15, 2023
@felixfontein felixfontein deleted the patchback/backports/stable-7/8801463575d9b3017154e091916e400148b58615/pr-6648 branch June 15, 2023 06:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature This issue/PR relates to a feature request integration tests/integration module module new_contributor Help guide this first time contributor plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants