PR#389: Add support for multiple LDAP queries -

no initial comment

There is always one host. I don't see a reason why multiple hosts would be needed.

It would be also faster if ldap.initialize(ldap_host) is called only once in waiverdb request.

Better to have a single new option with both base and filter values. E.g.:

LDAP_SEARCHES = [
    {
        "base": "ou=Groups,dc=example,dc=com",
        "filter": "(memberUid={user})"},
    },
    # ...
]

You would just need to handle the backwards compatibility elsewhere.

If you check the set(group_membership) & set(allowed_groups) here, you can avoid doing any following LDAP queries.

1 new commit added

Sep 23, 2020

Updated

Pull-Request has been merged by vmaljulin

Oct 2, 2020

Assignee

None

Tags

No Tags