Skip to content

Support custom GrantedAuthorities strategy for ActiveDirectoryLdapAuthenticationProvider #4390

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

Closed
mat013 opened this issue Jun 13, 2017 · 1 comment
Labels
in: ldap An issue in spring-security-ldap status: duplicate A duplicate of another issue type: enhancement A general enhancement

Comments

@mat013
Copy link

mat013 commented Jun 13, 2017

Summary

In connection with using ActiveDirectoryLdapAuthenticationProvider it is only possible to extract memberOf in the loadUserAuthorities. In our case we would have liked to extract some other attributes in particular the department and username. However it has not been possible to extend the ActiveDirectoryLdapAuthenticationProvider because the class it is final as it is now. I tried to see if there was some other good alternatives but nothing came up, so what I ended up with was copying the class and modifying it.

But rather than extending maybe it would be beneficial to let it be possible to provide a list of extractors to an instance of an AuthenticationProvider and then when loadAuthorities is invoked all extractors are invoked and the result of all the invocation is a merged list of all GrantedAuthorities.

Version

4.2.2.RELEASE

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label May 7, 2019
@jzheaux jzheaux changed the title Making it possible to provide alternative GrantedAuthorities for an AuthenticationProvider Support custom GrantedAuthorities strategy for ActiveDirectoryLdapAuthenticationProvider Jun 6, 2024
@jzheaux jzheaux assigned jzheaux and unassigned jzheaux Jun 6, 2024
@jzheaux jzheaux added in: ldap An issue in spring-security-ldap type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Jun 6, 2024
@jzheaux jzheaux added this to the 6.4.x milestone Jun 6, 2024
@jzheaux jzheaux added the status: duplicate A duplicate of another issue label Jun 6, 2024
@jzheaux
Copy link
Contributor

jzheaux commented Jun 6, 2024

Thanks for the report, @mat013, and I'm sorry for such a delay in responding. This was added in 6.3 as part of #4490 and so I'm going to close this as a duplicate of that.

@jzheaux jzheaux closed this as completed Jun 6, 2024
@jzheaux jzheaux removed this from the 6.4.x milestone Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: ldap An issue in spring-security-ldap status: duplicate A duplicate of another issue type: enhancement A general enhancement
Projects
None yet
Development

No branches or pull requests

3 participants