You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
defget_user_attributes(self, conn, userdn):
attrs= {}
ifself.auth_state_attributes:
found=conn.search(
search_base=userdn,
search_scope=ldap3.SUBTREE,
search_filter="(objectClass=*)",
attributes=self.auth_state_attributes,
)
# FIXME: Handle situations with multiple entries below or comment# why its not important to do.#iffound:
attrs=conn.entries[0].entry_attributes_as_dictreturnattrs
Maybe we should throw an error, same as in resolve_username? If there's a possibility of the entries corresponding to different Identities this implies a change in the LDAP server could lead to a different ordering of responses, resulting in a user gaining access to another user's account.
If it's two entries for the same user we still need to understand what the difference is, in case some attributes are different which could lead to inconsistent configuration of the singleuser server.
The text was updated successfully, but these errors were encountered:
Extracted from #294 (comment)
Regarding this code:
@manics wrote:
The text was updated successfully, but these errors were encountered: