mindol1004/spring-session
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix SessionRepository setHttpSessionStrategy Validationi

Browse Source 
Fixes gh-224
This commit is contained in:
james
2015-06-12 17:15:17 +02:00
committed by Rob Winch
parent c3c6215a24
commit 3b0dd129ba
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
spring-session/src/main/java/org/springframework/session/web/http/SessionRepositoryFilter.java
View File

@@ -77,7 +77,7 @@ public class SessionRepositoryFilter<S extends ExpiringSession> extends OncePerR
*/
public SessionRepositoryFilter(SessionRepository<S> sessionRepository) {
if(sessionRepository == null) {
throw new IllegalArgumentException("SessionRepository cannot be null");
throw new IllegalArgumentException("sessionRepository cannot be null");
}
this.sessionRepository = sessionRepository;
}
@@ -88,8 +88,8 @@ public class SessionRepositoryFilter<S extends ExpiringSession> extends OncePerR
* @param httpSessionStrategy the {@link HttpSessionStrategy} to use. Cannot be null.
*/
public void setHttpSessionStrategy(HttpSessionStrategy httpSessionStrategy) {
if(sessionRepository == null) {
throw new IllegalArgumentException("httpSessionIdStrategy cannot be null");
if(httpSessionStrategy == null) {
throw new IllegalArgumentException("httpSessionStrategy cannot be null");
}
this.httpSessionStrategy = new MultiHttpSessionStrategyAdapter(httpSessionStrategy);
}
@@ -100,8 +100,8 @@ public class SessionRepositoryFilter<S extends ExpiringSession> extends OncePerR
* @param httpSessionStrategy the {@link MultiHttpSessionStrategy} to use. Cannot be null.
*/
public void setHttpSessionStrategy(MultiHttpSessionStrategy httpSessionStrategy) {
if(sessionRepository == null) {
throw new IllegalArgumentException("httpSessionIdStrategy cannot be null");
if(httpSessionStrategy == null) {
throw new IllegalArgumentException("httpSessionStrategy cannot be null");
}
this.httpSessionStrategy = httpSessionStrategy;
}