expressa 2.0.3 → 2.0.4

package/controllers/collections.js

@@ -172,6 +172,7 @@ exports.replaceById = async function (req) {
     oldDoc = await req.db[req.params.collection].get(req.params.id)
   } catch (error) {
     // happens when new documents are created via a PUT
+    // todo: use setDocumentOwner
     oldDoc = { meta: { owner: req.user._id } }
   }
   const data = req.body
@@ -211,6 +212,11 @@ exports.updateById = async function (req) {
       doc.meta.owner_collection = ownerCollection
     }
   }
+  if (doc._id) {
+    doc._id = doc._id.trim()
+  } else {
+    throw new util.ApiError(400, '_id property is required')
+  }
   req.body = doc
   await util.notify('put', req, req.params.collection, doc)
   await req.db[req.params.collection].update(req.params.id, req.body)

package/index.js

@@ -114,6 +114,7 @@ module.exports.api = function (settings) {
       return util.getPgPool(router.settings.postgresql_uri)
     },
     doLogin: auth.doLogin,
+    createHash: auth.createHash,
     ...util,
   }
   router.eventListeners = {}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "expressa",
-  "version": "2.0.3",
+  "version": "2.0.4",
   "description": "API framework using JSON schemas",
   "main": "index.js",
   "repository": "https://github.com/thomas4019/expressa",

package/util.js

@@ -338,7 +338,9 @@ exports.generateDocumentId = function generateDocumentId() {
 
 exports.addIdIfMissing = function addIdIfMissing (document) {
   if (!document._id) {
-    document._id = exports.generateDocumentId()
+    document._id = exports.generateDocumentId().trim()
+  } else {
+    document._id = document._id.trim()
   }
 }