Fixed - RedissonSession.setAttribute method doesn't check the value for null...

Fixed - RedissonSession.setAttribute method doesn't check the value for null #2187 (thanks to jchobantonov)

redisson-tomcat/redisson-tomcat-6/src/main/java/org/redisson/tomcat/RedissonSession.java

@@ -281,7 +281,10 @@ public class RedissonSession extends StandardSession {
     public void setAttribute(String name, Object value, boolean notify) {
         super.setAttribute(name, value, notify);
         
-        if (updateMode == UpdateMode.DEFAULT && map != null && value != null) {
+        if (value == null) {
+            return;
+        }
+        if (updateMode == UpdateMode.DEFAULT && map != null) {
             fastPut(name, value);
         }
         if (updateMode == UpdateMode.AFTER_REQUEST) {

redisson-tomcat/redisson-tomcat-7/src/main/java/org/redisson/tomcat/RedissonSession.java

@@ -281,7 +281,10 @@ public class RedissonSession extends StandardSession {
     public void setAttribute(String name, Object value, boolean notify) {
         super.setAttribute(name, value, notify);
         
-        if (updateMode == UpdateMode.DEFAULT && map != null && value != null) {
+        if (value == null) {
+            return;
+        }
+        if (updateMode == UpdateMode.DEFAULT && map != null) {
             fastPut(name, value);
         }
         if (updateMode == UpdateMode.AFTER_REQUEST) {

redisson-tomcat/redisson-tomcat-8/src/main/java/org/redisson/tomcat/RedissonSession.java

@@ -280,8 +280,11 @@ public class RedissonSession extends StandardSession {
     @Override
     public void setAttribute(String name, Object value, boolean notify) {
         super.setAttribute(name, value, notify);
-        
-        if (updateMode == UpdateMode.DEFAULT && map != null && value != null) {
+
+        if (value == null) {
+            return;
+        }
+        if (updateMode == UpdateMode.DEFAULT && map != null) {
             fastPut(name, value);
         }
         if (updateMode == UpdateMode.AFTER_REQUEST) {

redisson-tomcat/redisson-tomcat-9/src/main/java/org/redisson/tomcat/RedissonSession.java

@@ -281,7 +281,10 @@ public class RedissonSession extends StandardSession {
     public void setAttribute(String name, Object value, boolean notify) {
         super.setAttribute(name, value, notify);
         
-        if (updateMode == UpdateMode.DEFAULT && map != null && value != null) {
+        if (value == null) {
+            return;
+        }
+        if (updateMode == UpdateMode.DEFAULT && map != null) {
             fastPut(name, value);
         }
         if (updateMode == UpdateMode.AFTER_REQUEST) {