fix encrypting new notes

src/entities/entity.js

@@ -17,9 +17,7 @@ class Entity {
     }
 
     beforeSaving() {
-        if (!this[this.constructor.primaryKeyName]) {
+        this.generateIdIfNecessary();
-            this[this.constructor.primaryKeyName] = utils.newEntityId();
-        }
 
         const origHash = this.hash;
 
@@ -28,6 +26,12 @@ class Entity {
         this.isChanged = origHash !== this.hash;
     }
 
+    generateIdIfNecessary() {
+        if (!this[this.constructor.primaryKeyName]) {
+            this[this.constructor.primaryKeyName] = utils.newEntityId();
+        }
+    }
+
     generateHash() {
         let contentToHash = "";
 

src/entities/note.js

@@ -513,6 +513,9 @@ class Note extends Entity {
             this.content = JSON.stringify(this.jsonContent, null, '\t');
         }
 
+        // we do this here because encryption needs the note ID for the IV
+        this.generateIdIfNecessary();
+
         if (this.isProtected) {
             protectedSessionService.encryptNote(this);
         }

src/routes/routes.js

@@ -84,7 +84,7 @@ function route(method, path, middleware, routeHandler, resultHandler, transactio
             }
         }
         catch (e) {
-            log.info(`${method} ${path} threw exception: ` + e.stack);
+            log.error(`${method} ${path} threw exception: ` + e.stack);
 
             res.sendStatus(500);
         }

src/services/data_encryption.js

@@ -85,10 +85,18 @@ function decryptString(dataKey, iv, cipherText) {
 }
 
 function noteTitleIv(iv) {
+    if (!iv) {
+        throw new Error("Empty iv!");
+    }
+
     return "0" + iv;
 }
 
 function noteContentIv(iv) {
+    if (!iv) {
+        throw new Error("Empty iv!");
+    }
+
     return "1" + iv;
 }