search

vt-middleware / ldaptive

A simple, extensible Java API for interacting with LDAP servers
56 stars 26 forks source link

Add a ResultHandler that joins all its entries (JoinEntriesResultHand… #191

Closed elmendavies closed 3 years ago

elmendavies commented 3 years ago

Dear all, I have added a search handler that uses the SearchResults.merge() function to join (or merge) all the entries by attribute name. I will continue adding the functionality to CAS.

The use case is to collect emails belonging to the same person with the search handler.

The person (the authenticated entry in CAS) is like this:

dn: person:1234,o=people,dc=... person: 1234 ...

The email entries are like this:

dn: my@mail.com,dc=... mail:, my@mail.com owner: person:1234,o=people,dc=... ...

Do you need further details or add any unit tests?

Bests regards, Miguel

elmendavies commented 3 years ago

Dear Daniel, I have added a test, a similar one to the one for the SortHandler. Best regards

dfish3r commented 3 years ago

Looks good, thanks for the contribution! Extra props for using Simpson test data.

elmendavies commented 3 years ago

Dear Daniel, Any idea about dates for this changes to be released? Best regards, Miguel

dfish3r commented 3 years ago

Planning for a release this month, there are a few other changes in flight. How soon do you need it?

elmendavies commented 3 years ago

It is good. Thank you