[jira] [Updated] (SHIRO-751) SimplePrincipalMap and SimplePrincipalCollection throw different exceptions for the same problem

["Francois Papon (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/SHIRO-751?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Francois Papon updated SHIRO-751:
---------------------------------
    Fix Version/s: 1.6.0

> SimplePrincipalMap and SimplePrincipalCollection throw different exceptions for the same problem
> ------------------------------------------------------------------------------------------------
>
>                 Key: SHIRO-751
>                 URL: https://issues.apache.org/jira/browse/SHIRO-751
>             Project: Shiro
>          Issue Type: Bug
>            Reporter: Hao Zhong
>            Priority: Major
>             Fix For: 1.6.0
>
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> When realname is null, SimplePrincipalCollection throws IllegalArgumentException:
>  
> {code:java}
> public void add(Object principal, String realmName) {
>         if (realmName == null) {
>             throw new IllegalArgumentException("realmName argument cannot be null.");
>         }
>         if (principal == null) {
>             throw new IllegalArgumentException("principal argument cannot be null.");
>         }
>         this.cachedToString = null;
>         getPrincipalsLazy(realmName).add(principal);
>     }
> {code}
> For the same problem, SimplePrincipalMap throws NullPointerException:
>  
> {code:java}
> public Object setRealmPrincipal(String realmName, String principalName, Object principal) { 
>  if (realmName == null) { 
>    throw new NullPointerException("realmName argument cannot be null."); 
>  }
>  ...}
>  
> {code}
> IllegalArgumentException seems to be more reasonable, and my code currently catches IllegalArgumentException for this type of the problem. Can the SimplePrincipalMap  be modified to throw IllegalArgumentException?



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