patch infinite session expiration bug

Author: nfawcett

spring-session-data-redis/src/main/java/org/springframework/session/data/redis/RedisIndexedSessionRepository.java

@@ -929,9 +929,12 @@ private void saveDelta() {
 
 			createShadowKey(sessionExpireInSeconds);
 
-			long fiveMinutesAfterExpires = sessionExpireInSeconds + TimeUnit.MINUTES.toSeconds(5);
-			RedisIndexedSessionRepository.this.sessionRedisOperations.boundHashOps(getSessionKey(getId()))
-				.expire(fiveMinutesAfterExpires, TimeUnit.SECONDS);
+			if (sessionExpireInSeconds > 0) {
+				long fiveMinutesAfterExpires = sessionExpireInSeconds + TimeUnit.MINUTES.toSeconds(5);
+				RedisIndexedSessionRepository.this.sessionRedisOperations.boundHashOps(getSessionKey(getId()))
+					.expire(fiveMinutesAfterExpires, TimeUnit.SECONDS);
+			}
+
 
 			RedisIndexedSessionRepository.this.expirationStore.save(this);
 			this.delta = new HashMap<>(this.delta.size());
@@ -1036,8 +1039,11 @@ public void save(RedisSession session) {
 
 			String expirationsKey = getExpirationsKey(toExpire);
 			long sessionExpireInSeconds = session.getMaxInactiveInterval().getSeconds();
-			long fiveMinutesAfterExpires = sessionExpireInSeconds + TimeUnit.MINUTES.toSeconds(5);
-			this.redis.boundSetOps(expirationsKey).expire(fiveMinutesAfterExpires, TimeUnit.SECONDS);
+
+			if (sessionExpireInSeconds > 0) {
+				long fiveMinutesAfterExpires = sessionExpireInSeconds + TimeUnit.MINUTES.toSeconds(5);
+				this.redis.boundSetOps(expirationsKey).expire(fiveMinutesAfterExpires, TimeUnit.SECONDS);
+			}
 
 			String expireKey = getExpirationKey(toExpire);
 			BoundSetOperations<String, Object> expireOperations = this.redis.boundSetOps(expireKey);