You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@shiro.apache.org by "YangL (JIRA)" <ji...@apache.org> on 2011/06/09 16:03:58 UTC
[jira] [Closed] (SHIRO-303) DefaultWebSecurityManager
isHttpSessionMode() may be error
[ https://issues.apache.org/jira/browse/SHIRO-303?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
YangL closed SHIRO-303.
-----------------------
Resolution: Not A Problem
> DefaultWebSecurityManager isHttpSessionMode() may be error
> ----------------------------------------------------------
>
> Key: SHIRO-303
> URL: https://issues.apache.org/jira/browse/SHIRO-303
> Project: Shiro
> Issue Type: Bug
> Components: Web
> Affects Versions: 1.1.0
> Environment: isHttpSessionMode() in org.apache.shiro.web.mgt.DefaultWebSecurityManager.java
> Reporter: YangL
> Labels: build
> Fix For: 1.2.0
>
> Original Estimate: 12h
> Remaining Estimate: 12h
>
> /**
> * @since 1.0
> */
> public boolean isHttpSessionMode() {
> return this.sessionMode == null || this.sessionMode.equals(HTTP_SESSION_MODE);
> }
> ---------------------------------------------------------------------------------------
> may be error,and i think this method should as bellow:
> public boolean isHttpSessionMode() {
> String sm = getSessionMode();
> return (sm!= null && sm.equals(HTTP_SESSION_MODE));
> }
>
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira