[ https://issues.apache.org/jira/browse/FTPSERVER-185?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12635707#action_12635707 ]
David Latorre commented on FTPSERVER-185: ----------------------------------------- sorry but I can't see what's your use-case here. Can you comment on why you need your own User implementation (not extending BaseUser!) and how you are implementing it so you can respond to "AuthorizationRequests" but you don't have a list of Authorities you can return. > Methods User#getAuthorities() is not used and should removed from the > interface > ------------------------------------------------------------------------------- > > Key: FTPSERVER-185 > URL: https://issues.apache.org/jira/browse/FTPSERVER-185 > Project: FtpServer > Issue Type: Bug > Components: Ftplets > Reporter: Andrea Francia > > As far I understand the User interface should specify how the User > implementations should communicates with the ftpserver. > The ftpserver doesn't known directly if a user is should be authorized to > perform a specific action but it delegate this decisione to the User > implementation. > As I can see from the source code the method for determining if a user can > perform a specific action is: > public interface User { > AuthorizationRequest authorize(AuthorizationRequest request); > ... > } > So I don't see the purpose of put in the interface these methods: > Authority[] getAuthorities(); > Authority[] getAuthorities(Class<? extends Authority> clazz); > These methods are not used by the ftpserver so they should not go in the > interface. > The interface beetween two entities should be keep simple as possible. > The getAutorirhies() methods are used only by the specific implementation of > User named BaseUser, another implementation of User should be free to choose > another method for handling permissions. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.