Fix email formatting (I think)
[vchess.git] / server / models / User.js
index cf4c529..70d8b52 100644 (file)
@@ -1,7 +1,7 @@
 var db = require("../utils/database");
-var maild = require("../utils/mailer.js");
 var genToken = require("../utils/tokenGenerator");
 var params = require("../config/parameters");
+var sendEmail = require('../utils/mailer');
 
 /*
  * Structure:
@@ -12,108 +12,163 @@ var params = require("../config/parameters");
  *   loginTime: datetime (validity)
  *   sessionToken: token in cookies for authentication
  *   notify: boolean (send email notifications for corr games)
+ *   created: datetime
  */
 
 const UserModel =
 {
-       checkNameEmail: function(o)
-       {
-               if (typeof o.name === "string")
-               {
-                       if (o.name.length == 0)
-                               return "Empty name";
-                       if (!o.name.match(/^[\w]+$/))
-                               return "Bad characters in name";
-               }
-               if (typeof o.email === "string")
-               {
-                       if (o.email.length == 0)
-                               return "Empty email";
-                       if (!o.email.match(/^[\w.+-]+@[\w.+-]+$/))
-                               return "Bad characters in email";
-               }
-       },
+  checkNameEmail: function(o)
+  {
+    if (typeof o.name === "string")
+    {
+      if (o.name.length == 0)
+        return "Empty name";
+      if (!o.name.match(/^[\w]+$/))
+        return "Bad characters in name";
+    }
+    if (typeof o.email === "string")
+    {
+      if (o.email.length == 0)
+        return "Empty email";
+      if (!o.email.match(/^[\w.+-]+@[\w.+-]+$/))
+        return "Bad characters in email";
+    }
+    return ""; //NOTE: not required, but more consistent... (?!)
+  },
 
-       // NOTE: parameters are already cleaned (in controller), thus no sanitization here
-       create: function(name, email, notify, callback)
-       {
-               db.serialize(function() {
-                       const insertQuery =
-                               "INSERT INTO Users " +
-                               "(name, email, notify) VALUES " +
-                               "('" + name + "', '" + email + "', " + notify + ")";
-                       db.run(insertQuery, err => {
-                               if (!!err)
-                                       return callback(err);
-                               db.get("SELECT last_insert_rowid() AS rowid", callback);
-                       });
-               });
-       },
+  // NOTE: parameters are already cleaned (in controller), thus no sanitization here
+  create: function(name, email, notify, callback)
+  {
+    db.serialize(function() {
+      const insertQuery =
+        "INSERT INTO Users " +
+        "(name, email, notify, created) VALUES " +
+        "('" + name + "', '" + email + "', " + notify + "," + Date.now() + ")";
+      db.run(insertQuery, err => {
+        if (!!err)
+          return callback(err);
+        db.get("SELECT last_insert_rowid() AS rowid", callback);
+      });
+    });
+  },
 
-       // Find one user (by id, name, email, or token)
-       getOne: function(by, value, cb)
-       {
-               const delimiter = (typeof value === "string" ? "'" : "");
-               db.serialize(function() {
-                       const query =
-                               "SELECT * " +
-                               "FROM Users " +
-                               "WHERE " + by + " = " + delimiter + value + delimiter;
-                       db.get(query, cb);
-               });
-       },
+  // Find one user (by id, name, email, or token)
+  getOne: function(by, value, cb)
+  {
+    const delimiter = (typeof value === "string" ? "'" : "");
+    db.serialize(function() {
+      const query =
+        "SELECT * " +
+        "FROM Users " +
+        "WHERE " + by + " = " + delimiter + value + delimiter;
+      db.get(query, cb);
+    });
+  },
 
-       /////////
-       // MODIFY
+  getByIds: function(ids, cb) {
+    db.serialize(function() {
+      const query =
+        "SELECT id, name " +
+        "FROM Users " +
+        "WHERE id IN (" + ids + ")";
+      db.all(query, cb);
+    });
+  },
 
-       setLoginToken: function(token, uid, cb)
-       {
-               db.serialize(function() {
-                       const query =
-                               "UPDATE Users " +
-                               "SET loginToken = '" + token + "', loginTime = " + Date.now() + " " +
-                               "WHERE id = " + uid;
-                       db.run(query, cb);
-               });
-       },
+  /////////
+  // MODIFY
 
-       // Set session token only if empty (first login)
-       // TODO: weaker security (but avoid to re-login everywhere after each logout)
-       trySetSessionToken: function(uid, cb)
-       {
-               // Also empty the login token to invalidate future attempts
-               db.serialize(function() {
-                       const querySessionToken =
-                               "SELECT sessionToken " +
-                               "FROM Users " +
-                               "WHERE id = " + uid;
-                       db.get(querySessionToken, (err,ret) => {
-                               if (!!err)
-                                       return cb(err);
-                               const token = ret.sessionToken || genToken(params.token.length);
-                               const queryUpdate =
-                                       "UPDATE Users " +
-                                       "SET loginToken = NULL" +
-                                       (!ret.sessionToken ? (", sessionToken = '" + token + "'") : "") + " " +
-                                       "WHERE id = " + uid;
-                               db.run(queryUpdate);
-                               cb(null, token);
-                       });
-               });
-       },
+  setLoginToken: function(token, uid, cb)
+  {
+    db.serialize(function() {
+      const query =
+        "UPDATE Users " +
+        "SET loginToken = '" + token + "', loginTime = " + Date.now() + " " +
+        "WHERE id = " + uid;
+      db.run(query, cb);
+    });
+  },
 
-       updateSettings: function(user, cb)
-       {
-               db.serialize(function() {
-                       const query =
-                               "UPDATE Users " +
-                               "SET name = '" + user.name + "'" +
-                               ", email = '" + user.email + "'" +
-                               ", notify = " + user.notify + " " +
-                               "WHERE id = " + user.id;
-                       db.run(query, cb);
-               });
-       },
+  // Set session token only if empty (first login)
+  // NOTE: weaker security (but avoid to re-login everywhere after each logout)
+  // TODO: option would be to reset all tokens periodically, e.g. every 3 months
+  trySetSessionToken: function(uid, cb)
+  {
+    // Also empty the login token to invalidate future attempts
+    db.serialize(function() {
+      const querySessionToken =
+        "SELECT sessionToken " +
+        "FROM Users " +
+        "WHERE id = " + uid;
+      db.get(querySessionToken, (err,ret) => {
+        if (!!err)
+          return cb(err);
+        const token = ret.sessionToken || genToken(params.token.length);
+        const queryUpdate =
+          "UPDATE Users " +
+          "SET loginToken = NULL" +
+          (!ret.sessionToken ? (", sessionToken = '" + token + "'") : "") + " " +
+          "WHERE id = " + uid;
+        db.run(queryUpdate);
+        cb(null, token);
+      });
+    });
+  },
+
+  updateSettings: function(user, cb)
+  {
+    db.serialize(function() {
+      const query =
+        "UPDATE Users " +
+        "SET name = '" + user.name + "'" +
+        ", email = '" + user.email + "'" +
+        ", notify = " + user.notify + " " +
+        "WHERE id = " + user.id;
+      db.run(query, cb);
+    });
+  },
+
+  /////////////////
+  // NOTIFICATIONS
+
+  notify: function(user, message)
+  {
+    const subject = "vchess.club - notification";
+    const body = "Hello " + user.name + "!" + `
+` + message;
+    sendEmail(params.mail.noreply, user.email, subject, body);
+  },
+
+  tryNotify: function(id, message)
+  {
+    UserModel.getOne("id", id, (err,user) => {
+      if (!err || !user.notify)
+        return; //NOTE: error is ignored here
+      UserModel.notify(user, message);
+    });
+  },
+
+  ////////////
+  // CLEANING
+
+  cleanUsersDb: function()
+  {
+    const tsNow = Date.now();
+    // 86400000 = 24 hours in milliseconds
+    const day = 86400000;
+    db.serialize(function() {
+      const query =
+        "SELECT id, sessionToken, created " +
+        "FROM Users";
+      db.all(query, (err, users) => {
+        users.forEach(u => {
+          // Remove unlogged users for >1 day
+          if (!u.sessionToken && tsNow - u.created > day)
+            db.run("DELETE FROM Users WHERE id = " + u.id);
+        });
+      });
+    });
+  },
 }
 
 module.exports = UserModel;