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

SEC-1483: Change org.springframework.security.core.userdetails.User constructor to accept Collection<? extends GrantedAuthority> #1724

Closed
spring-issuemaster opened this issue May 20, 2010 · 2 comments

Comments

@spring-issuemaster
Copy link

@spring-issuemaster spring-issuemaster commented May 20, 2010

Ronan Daly (Migrated from SEC-1483) said:

Currently the User constructor takes a Collection as a parameter. If it were to take Collection<? extends GrantedAuthority> it would mean that users would not have to implement a special method just to generate the collection as that type. The change would be trivial.

@spring-issuemaster

This comment has been minimized.

Copy link
Author

@spring-issuemaster spring-issuemaster commented May 21, 2010

Luke Taylor said:

Added to 3.0 and master branches.

@spring-issuemaster

This comment has been minimized.

Copy link
Author

@spring-issuemaster spring-issuemaster commented Jul 7, 2010

Johnathon said:

It would seem that UserDetails itself got left behind in this modification, as it would seem to make sense to also allow any implementing classes of UserDetails to accept <? extends GrantedAuthority> on the getAuthorities method definition as well... Would it make sens to reopen this task, or create a separate one?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.