Merge pull request #4176 from Coduz/fix-npeOnPreUpdateIfNoSession

🐛 [Core] Fixed handling of null KapuaSession on prePersist and preUpdate operation in KapuaEntity
eclipse-kapua · Jan 20, 2025 · 82b5b18 · 82b5b18
2 parents 88f427b + 425f96b
commit 82b5b18
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/commons/src/main/java/org/eclipse/kapua/commons/model/AbstractKapuaEntity.java b/commons/src/main/java/org/eclipse/kapua/commons/model/AbstractKapuaEntity.java
@@ -165,7 +165,9 @@ public void setCreatedBy(KapuaId createdBy) {
     @PrePersist
     protected void prePersistsAction() {
         setId(new KapuaEid(IdGenerator.generate()));
-        setCreatedBy(KapuaSecurityUtils.getSession().getUserId());
+        if (KapuaSecurityUtils.getSession() != null) {
+            setCreatedBy(KapuaSecurityUtils.getSession().getUserId());
+        }
         setCreatedOn(new Date());
     }
 }
diff --git a/commons/src/main/java/org/eclipse/kapua/commons/model/AbstractKapuaUpdatableEntity.java b/commons/src/main/java/org/eclipse/kapua/commons/model/AbstractKapuaUpdatableEntity.java
@@ -212,7 +212,9 @@ protected void prePersistsAction() {
      */
     @PreUpdate
     protected void preUpdateAction() {
-        setModifiedBy(KapuaSecurityUtils.getSession().getUserId());
+        if (KapuaSecurityUtils.getSession() != null) {
+            setModifiedBy(KapuaSecurityUtils.getSession().getUserId());
+        }
         setModifiedOn(new Date());
     }
 }