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

Optimization for multi-valued LDAP attributes such as groups uniqueMember #259

Closed
wants to merge 2 commits into from

Conversation

rouazana
Copy link
Contributor

See #255

Use smaller ADD and DELETE modifications instead of a huge REPLACE, at worst equivalent
@thperchi
Copy link

Hello,

I'm reaching out to ask about something regarding this pull request.

I am the operational lead for open source support at INTM Group, and we have been collaborating with EDF on the development of this feature. EDF approached us to make specific modifications, and we created a fork of the project to carry out these developments.

However, EDF wanted to present this contribution on behalf of their organization, as they are keen to demonstrate their commitment to the open-source community. Therefore, we would prefer the pull request to be submitted from EDF's official account.

I understand this may seem a bit inconvenient, but would it be possible to delete the current pull request and create a new one from EDF's account? This would allow us to honor our client's wishes and maintain good communication with the open-source community.

Thank you in advance for your understanding and cooperation. Please feel free to contact me if you have any further questions or concerns.

Best regards,

@coudot
Copy link
Member

coudot commented Apr 17, 2024

Hello @thperchi

this PR is still open so you can propose a new one and write in description that it replaces this one. We will the close it to work on the new PR.

@coudot
Copy link
Member

coudot commented May 28, 2024

Can we close this one as #285 replaces it?

@coudot coudot closed this Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Optimization for multi-valued LDAP attributes such as groups uniqueMember
4 participants