[ 
https://issues.apache.org/jira/browse/SHIRO-588?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17081212#comment-17081212
 ] 

Benjamin Marwell commented on SHIRO-588:
----------------------------------------

If this is targeted for 2.x, then there is no need for a new ldap realm. If 
missed the target version. 😉

> DefaultLdapRealm should extend AbstractLdapRealm
> ------------------------------------------------
>
>                 Key: SHIRO-588
>                 URL: https://issues.apache.org/jira/browse/SHIRO-588
>             Project: Shiro
>          Issue Type: Improvement
>            Reporter: Brian Demers
>            Priority: Major
>             Fix For: 2.0.0
>
>
> DefaultLdapRealm not extending AbstractLdapRealm is confusing. If there is a 
> reason for it, we should add that do the javadoc.
> From: 
> http://shiro-user.582556.n2.nabble.com/Confused-By-Class-Hierarchy-tc7581289.html
> Note: this might not be a breaking change, as the API looks simliar (but 
> needs another look)



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to