remove uuid, only use nanoid and u64

config/secret/index.js

@@ -1,4 +1,4 @@
-const { v4: uuid} = require('uuid');
+const { nanoid } = require('nanoid')
 const path = require('path');
 const fs = require('fs');
 
@@ -7,7 +7,7 @@ const secretFilePath = path.join((process.env.SECRET_DIRNAME ? process.env.SECRE
 try {
   module.exports = fs.readFileSync(secretFilePath).toString();
 } catch (_) {
-  const secret = uuid();
+  const secret = nanoid(128);
   fs.writeFileSync(secretFilePath, secret);
   module.exports = secret;
 }

package.json

@@ -31,7 +31,7 @@
     "passport-local": "^1.0.0",
     "pouchdb": "^7.2.2",
     "pug": "^3.0.0",
-    "uuid": "^8.3.1",
+    "u64": "^1.0.1",
     "yes-no": "^0.0.1"
   }
 }

routes/wishlist/index.js

@@ -2,7 +2,7 @@ const verifyAuth = require('../../middlewares/verifyAuth');
 const getProductName = require('get-product-name');
 const express = require('express');
 const config = require('../../config');
-const { v4: uuid} = require('uuid');
+const u64 = require('u64')
 
 const totals = wishlist => {
   let unpledged = 0;
@@ -70,6 +70,10 @@ module.exports = (db) => {
   });
 
   router.post('/:user', verifyAuth(), async (req, res) => {
+    if (!req.body.itemUrlOrName) {
+      req.flash('error', 'Item URL or Name is required')
+      return res.redirect(`/wishlist/${req.params.user}`)
+    }
     const potentialUrl = req.body.itemUrlOrName.split(' ').pop();
     const url = ValidURL(potentialUrl);
     const item = {};
@@ -85,10 +89,15 @@ module.exports = (db) => {
     item.note = req.body.note;
     if (url) item.url = url;
     if (!url) item.name = req.body.itemUrlOrName
-    item.id = uuid();
+    item.id = u64.encode(new Date().getTime().toString());
     const doc = await db.get(req.params.user);
     doc.wishlist.push(item);
+    try {
       await db.put(doc);
+    } catch {
+      req.flash('error', 'Items are being added too quickly. Please try again.')
+      return res.redirect(`/wishlist/${req.params.user}`)
+    }
     req.flash(
       'success',
       (