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

Remove call to api.Backend.ldap2.disconnect() #311

Merged
merged 1 commit into from
Nov 9, 2023

Conversation

rcritten
Copy link
Collaborator

@rcritten rcritten commented Nov 9, 2023

This was added while I was testing the IPA LDAP client cache performance. By disconnecting a summary of the cache is logged. I never intended it remain in the code.

@flo-renaud flo-renaud self-assigned this Nov 9, 2023
@flo-renaud
Copy link
Contributor

@rcritten thanks for the patch, works for me. It properly fixes #310, you may want to add a reference to the issue in your commit msg.

@flo-renaud flo-renaud added the ack label Nov 9, 2023
This was added while I was testing the IPA LDAP client
cache performance. By disconnecting a summary of the cache is
logged. I never intended it remain in the code.

Fixes: freeipa#310

Signed-off-by: Rob Crittenden <rcritten@redhat.com>
@rcritten
Copy link
Collaborator Author

rcritten commented Nov 9, 2023

Missing ticket reference added.

@rcritten rcritten merged commit a56f1f8 into freeipa:master Nov 9, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants