439753 ConstraintSecurityHandler has dead code for processing constraints
This commit is contained in:
parent
a89419fd3f
commit
3975d1170a
|
@ -511,21 +511,6 @@ public class ConstraintSecurityHandler extends SecurityHandler implements Constr
|
|||
mappings.put(ALL_METHODS,roleInfo);
|
||||
}
|
||||
}
|
||||
else
|
||||
{
|
||||
//combine with any entry that covers all methods
|
||||
if (httpMethod == null)
|
||||
{
|
||||
for (Map.Entry<String, RoleInfo> entry : mappings.entrySet())
|
||||
{
|
||||
if (entry.getKey() != null)
|
||||
{
|
||||
RoleInfo specific = entry.getValue();
|
||||
specific.combine(roleInfo);
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
/* ------------------------------------------------------------ */
|
||||
|
|
|
@ -441,7 +441,7 @@ public class ConstraintTest
|
|||
|
||||
Constraint constraint6 = new Constraint();
|
||||
constraint6.setAuthenticate(true);
|
||||
constraint6.setName("omit POST and GET");
|
||||
constraint6.setName("omit HEAD and GET");
|
||||
constraint6.setRoles(new String[]{"user"});
|
||||
ConstraintMapping mapping6 = new ConstraintMapping();
|
||||
mapping6.setPathSpec("/omit/*");
|
||||
|
|
Loading…
Reference in New Issue