Use the new module and uninstall shortid

server/lib/message-sender.js

@@ -24,7 +24,7 @@ const blacklist = require('../models/blacklist');
 const libmime = require('libmime');
 const { enforce, hashEmail } = require('./helpers');
 const senders = require('./senders');
-const shortid = require('shortid');
+const shortid = require('./shortid');
 
 const MessageType = {
     REGULAR: 0,

server/models/campaigns.js

@@ -4,7 +4,7 @@ const knex = require('../lib/knex');
 const hasher = require('node-object-hash')();
 const dtHelpers = require('../lib/dt-helpers');
 const interoperableErrors = require('../../shared/interoperable-errors');
-const shortid = require('shortid');
+const shortid = require('../lib/shortid');
 const { enforce, filterObject } = require('../lib/helpers');
 const shares = require('./shares');
 const namespaceHelpers = require('../lib/namespace-helpers');
@@ -1163,4 +1163,4 @@ module.exports.getStatisticsOpened = getStatisticsOpened;
 module.exports.fetchRssCampaign = fetchRssCampaign;
 
 module.exports.testSend = testSend;
-module.exports.getRssPreview = getRssPreview;
+module.exports.getRssPreview = getRssPreview;

server/models/channels.js

@@ -4,7 +4,7 @@ const knex = require('../lib/knex');
 const hasher = require('node-object-hash')();
 const dtHelpers = require('../lib/dt-helpers');
 const interoperableErrors = require('../../shared/interoperable-errors');
-const shortid = require('shortid');
+const shortid = require('../lib/shortid');
 const { enforce, filterObject } = require('../lib/helpers');
 const shares = require('./shares');
 const namespaceHelpers = require('../lib/namespace-helpers');

server/models/confirmations.js

@@ -1,7 +1,7 @@
 'use strict';
 
 const knex = require('../lib/knex');
-const shortid = require('shortid');
+const shortid = require('../lib/shortid');
 
 async function addConfirmation(listId, action, ip, data) {
     const cid = shortid.generate();

server/models/fields.js

@@ -7,7 +7,7 @@ const dtHelpers = require('../lib/dt-helpers');
 const interoperableErrors = require('../../shared/interoperable-errors');
 const shares = require('./shares');
 const validators = require('../../shared/validators');
-const shortid = require('shortid');
+const shortid = require('../lib/shortid');
 const slugify = require('slugify');
 const segments = require('./segments');
 const { formatDate, formatBirthday, parseDate, parseBirthday } = require('../../shared/date');

server/models/links.js

@@ -10,7 +10,7 @@ const uaParser = require('device');
 const he = require('he');
 const { getPublicUrl } = require('../lib/urls');
 const tools = require('../lib/tools');
-const shortid = require('shortid');
+const shortid = require('../lib/shortid');
 const {enforce} = require('../lib/helpers');
 
 const LinkId = {

server/models/lists.js

@@ -3,7 +3,7 @@
 const knex = require('../lib/knex');
 const hasher = require('node-object-hash')();
 const dtHelpers = require('../lib/dt-helpers');
-const shortid = require('shortid');
+const shortid = require('../lib/shortid');
 const { enforce, filterObject } = require('../lib/helpers');
 const interoperableErrors = require('../../shared/interoperable-errors');
 const shares = require('./shares');

server/models/send-configurations.js

@@ -3,7 +3,7 @@
 const knex = require('../lib/knex');
 const hasher = require('node-object-hash')();
 const dtHelpers = require('../lib/dt-helpers');
-const shortid = require('shortid');
+const shortid = require('../lib/shortid');
 const { enforce, filterObject } = require('../lib/helpers');
 const interoperableErrors = require('../../shared/interoperable-errors');
 const shares = require('./shares');

server/models/subscriptions.js

@@ -3,7 +3,7 @@
 const config = require('../lib/config');
 const knex = require('../lib/knex');
 const hasher = require('node-object-hash')();
-const shortid = require('shortid');
+const shortid = require('../lib/shortid');
 const dtHelpers = require('../lib/dt-helpers');
 const interoperableErrors = require('../../shared/interoperable-errors');
 const shares = require('./shares');

server/package-lock.json

@@ -5038,9 +5038,9 @@
       "optional": true
     },
     "nanoid": {
-      "version": "2.1.7",
-      "resolved": "https://registry.npmjs.org/nanoid/-/nanoid-2.1.7.tgz",
-      "integrity": "sha512-fmS3qwDldm4bE01HCIRqNk+f255CNjnAoeV3Zzzv0KemObHKqYgirVaZA9DtKcjogicWjYcHkJs4D5A8CjnuVQ=="
+      "version": "3.1.12",
+      "resolved": "https://registry.npmjs.org/nanoid/-/nanoid-3.1.12.tgz",
+      "integrity": "sha512-1qstj9z5+x491jfiC4Nelk+f8XBad7LN20PmyWINJEMRSf3wcAjAWysw1qaA8z6NSKe2sjq1hRSDpBH5paCb6A=="
     },
     "nanomatch": {
       "version": "1.2.13",
@@ -12744,14 +12744,6 @@
       "resolved": "https://registry.npmjs.org/shebang-regex/-/shebang-regex-1.0.0.tgz",
       "integrity": "sha1-2kL0l0DAtC2yypcoVxyxkMmO/qM="
     },
-    "shortid": {
-      "version": "2.2.15",
-      "resolved": "https://registry.npmjs.org/shortid/-/shortid-2.2.15.tgz",
-      "integrity": "sha512-5EaCy2mx2Jgc/Fdn9uuDuNIIfWBpzY4XIlhoqtXF6qsf+/+SGZ+FxDdX/ZsMZiWupIWNqAEmiNY4RC+LSmCeOw==",
-      "requires": {
-        "nanoid": "^2.1.0"
-      }
-    },
     "sigmund": {
       "version": "1.0.1",
       "resolved": "https://registry.npmjs.org/sigmund/-/sigmund-1.0.1.tgz",

server/package.json

@@ -106,7 +106,6 @@
     "request": "^2.88.0",
     "request-promise": "^4.2.5",
     "serve-favicon": "^2.5.0",
-    "shortid": "^2.2.15",
     "slugify": "^1.3.6",
     "smtp-server": "^3.4.7",
     "tmp-promise": "^1.1.0",

server/setup/knex/fixes/fix-20190726150000_shorten_field_column_names.js

@@ -2,7 +2,7 @@
 
 const config = require('../config');
 const knex = require('../../../lib/knex');
-const shortid = require('shortid');
+const shortid = require('../../../shortid');
 const slugify = require('slugify');
 
 async function run() {
@@ -66,4 +66,4 @@ async function run() {
     process.exit();
 }
 
-run().catch(err => console.error(err));
+run().catch(err => console.error(err));

server/setup/knex/migrations/20170506102634_v1_to_v2.js

@@ -11,7 +11,7 @@ const { EntityVals: TriggerEntityVals, EventVals: TriggerEventVals } = require('
 const { SubscriptionSource } = require('../../../../shared/lists');
 const {DOMParser, XMLSerializer} = require('xmldom');
 const log = require('../../../lib/log');
-const shortid = require('shortid');
+const shortid = require('../../../lib/shortid');
 const slugify = require('slugify');
 
 const entityTypesAddNamespace = ['list', 'custom_form', 'template', 'campaign', 'report', 'report_template', 'user'];

server/test/e2e/tests/subscription.js

@@ -4,7 +4,7 @@
 
 const config = require('../lib/config');
 const { useCase, step, precondition, driver } = require('../lib/mocha-e2e');
-const shortid = require('shortid');
+const shortid = require('../lib/shortid');
 const expect = require('chai').expect;
 const createPage = require('../page-objects/subscription');
 const faker = require('faker');