fix(domaincache-and-domainaliases): ZMS-259 add wildcard address check when adding domain to mongodb cache (#859)

* add wildcard address check when adding domain to mongodb cache. insert alias domains into cache too

* domainaliases use insertMany instead of sequential insertOne

Author: NickOvt

lib/api/addresses.js

@@ -533,7 +533,10 @@ module.exports = (db, server, userHandler, settingsHandler) => {
             try {
                 r = await db.users.collection('addresses').insertOne(addressData);
                 try {
-                    await db.database.collection('domaincache').insertOne({ domain: addressData.address.split('@')[1] });
+                    const domain = addressData.address.split('@')[1];
+                    if (domain && !domain.includes('*')) {
+                        await db.users.collection('domaincache').insertOne({ domain });
+                    }
                 } catch {
                     // ignore
                 }
@@ -1731,7 +1734,10 @@ module.exports = (db, server, userHandler, settingsHandler) => {
                 r = await db.users.collection('addresses').insertOne(addressData);
 
                 try {
-                    await db.database.collection('domaincache').insertOne({ domain: addressData.address.split('@')[1] });
+                    const domain = addressData.address.split('@')[1];
+                    if (domain && !domain.includes('*')) {
+                        await db.users.collection('domaincache').insertOne({ domain });
+                    }
                 } catch {
                     // ignore
                 }

lib/api/domainaliases.js

@@ -262,6 +262,12 @@ module.exports = (db, server) => {
                     domain,
                     created: new Date()
                 });
+
+                try {
+                    await db.users.collection('domaincache').insertMany([{ domain: alias }, { domain }]);
+                } catch {
+                    // ignore
+                }
             } catch (err) {
                 res.status(500);
                 return res.json({

lib/user-handler.js

@@ -1611,7 +1611,10 @@ class UserHandler {
                 await this.users.collection('addresses').insertOne(addressData, { writeConcern: 'majority' });
 
                 try {
-                    await this.users.collection('domaincache').insertOne({ domain: addressData.address.split('@')[1] });
+                    const domain = addressData.address.split('@')[1];
+                    if (domain && !domain.includes('*')) {
+                        await this.users.collection('domaincache').insertOne({ domain });
+                    }
                 } catch {
                     // ignore
                 }
@@ -3684,7 +3687,10 @@ class UserHandler {
                 }
 
                 try {
-                    await this.users.collection('domaincache').insertOne({ domain: address.address.split('@')[1] });
+                    const domain = address.address.split('@')[1];
+                    if (domain && !domain.includes('*')) {
+                        await this.users.collection('domaincache').insertOne({ domain });
+                    }
                 } catch {
                     // ignore
                 }