fix: optimize lookup, bump deps

Author: niftylettuce

README.md

@@ -68,6 +68,6 @@ Read our [Terms](https://forwardemail.net/terms).
 [Business Source License 1.1](LICENSE) © [Niftylettuce, LLC.](https://niftylettuce.com/)
 
 
-## 
+##
 
 <a href="#"><img src="https://raw.githubusercontent.com/forwardemail/free-email-forwarding/master/media/footer.png" alt="#" /></a>

SELF-HOSTING.md

@@ -22,7 +22,7 @@ You'll also need the following dependencies installed:
 
 * [Redis][] (v4.x+) - this is a fast key-value store database used for rate-limiting and preventing spammers
 
-  > _NOTE_: You can pass `rateLimit: false` as an option to your `ForwardEmail` instance to disable the Redis requirement (e.g. `const forwardEmail = new ForwardEmail({ rateLimit: false });`
+  > *NOTE*: You can pass `rateLimit: false` as an option to your `ForwardEmail` instance to disable the Redis requirement (e.g. `const forwardEmail = new ForwardEmail({ rateLimit: false });`
 
   * Mac (via [brew][]): `brew install redis && brew services start redis`
   * Ubuntu:
@@ -64,7 +64,6 @@ You'll also need the following dependencies installed:
   sudo apt-get install libstemmer-dev libstemmer-tools
   ```
 
-
 * [authbind][] - for allowing non-root users to run on restricted ports
 
   * Ubuntu:
@@ -178,7 +177,7 @@ See the [app.js](app.js) and [ecosystem.json](ecosystem.json) files for more ins
 [Business Source License 1.1](LICENSE) © [Niftylettuce, LLC.](https://niftylettuce.com/)
 
 
-## 
+##
 
 [npm]: https://www.npmjs.com/
 

helpers/message-splitter.js

@@ -21,7 +21,9 @@ class MessageSplitter extends Transform {
     this.headerChunks = [];
     this.rawHeaders = false;
     this.dataBytes = 0;
-    this._maxBytes = (options.maxBytes && Number(options.maxBytes)) || Infinity;
+    this._maxBytes =
+      (options.maxBytes && Number(options.maxBytes)) ||
+      Number.POSITIVE_INFINITY;
     this.sizeExceeded = false;
   }
 

index.js

@@ -2410,6 +2410,11 @@ class ForwardEmail {
         );
     }
 
+    // check if we have a specific redirect and store global redirects (if any)
+    // get username from recipient email address
+    // (e.g. hello@niftylettuce.com => hello)
+    const username = this.parseUsername(address);
+
     if (verifications.length > 0) {
       if (verifications.length > 1)
         throw new CustomError(
@@ -2420,7 +2425,7 @@ class ForwardEmail {
       try {
         const { body } = await superagent
           .get(`${this.config.apiEndpoint}/v1/lookup`)
-          .query({ verification_record: verifications[0] })
+          .query({ domain, username, verification_record: verifications[0] })
           .set('Accept', 'json')
           .set('User-Agent', this.config.userAgent)
           .auth(this.config.apiSecrets[0])
@@ -2478,11 +2483,6 @@ class ForwardEmail {
     // store if we have a global redirect or not
     const globalForwardingAddresses = [];
 
-    // check if we have a specific redirect and store global redirects (if any)
-    // get username from recipient email address
-    // (e.g. hello@niftylettuce.com => hello)
-    const username = this.parseUsername(address);
-
     for (const element of addresses) {
       // convert addresses to lowercase
       const lowerCaseAddress = element.toLowerCase();

package.json

@@ -29,11 +29,11 @@
     "@ladjs/graceful": "^1.0.5",
     "@ladjs/proxy": "^2.0.3",
     "@ladjs/redis": "^1.0.7",
-    "@ladjs/shared-config": "^3.0.9",
-    "@slack/web-api": "^5.14.0",
+    "@ladjs/shared-config": "^3.0.10",
+    "@slack/web-api": "^6.0.0",
     "array-join-conjunction": "^1.0.0",
     "authheaders": "^2.0.2",
-    "axe": "^7.0.2",
+    "axe": "^8.0.0",
     "boolean": "^3.0.2",
     "bytes": "^3.1.0",
     "common-tags": "^1.8.0",
@@ -49,12 +49,12 @@
     "is-string-and-not-blank": "^0.0.2",
     "lodash": "^4.17.20",
     "mailparser": "^3.0.1",
-    "mailsplit": "^5.0.0",
+    "mailsplit": "^5.0.1",
     "ms": "^2.1.3",
     "mx-connect": "^1.2.0",
-    "nodemailer": "^6.4.16",
+    "nodemailer": "^6.4.17",
     "pify": "^5.0.0",
-    "pino": "^6.7.0",
+    "pino": "^6.11.0",
     "punycode": "^2.1.1",
     "ratelimiter": "^3.4.1",
     "re2": "^1.15.9",
@@ -72,21 +72,21 @@
   "devDependencies": {
     "@commitlint/cli": "^11.0.0",
     "@commitlint/config-conventional": "^11.0.0",
-    "ava": "^3.14.0",
+    "ava": "^3.15.0",
     "codecov": "^3.8.1",
     "cross-env": "^7.0.3",
-    "eslint": "^7.15.0",
+    "eslint": "^7.18.0",
     "eslint-config-xo-lass": "^1.0.4",
-    "fixpack": "^3.0.6",
-    "husky": "^4.3.5",
-    "ioredis": "^4.19.2",
+    "fixpack": "^4.0.0",
+    "husky": "^4.3.8",
+    "ioredis": "^4.19.4",
     "is-ci": "^2.0.0",
     "lint-staged": "^10.5.3",
     "nyc": "^15.1.0",
     "remark-cli": "^9.0.0",
     "remark-preset-github": "^4.0.1",
     "uuid": "^8.3.2",
-    "xo": "^0.36.1"
+    "xo": "^0.37.1"
   },
   "engines": {
     "node": ">=12"
@@ -157,7 +157,8 @@
       "config.js"
     ],
     "rules": {
-      "no-use-extend-native/no-use-extend-native": "off"
+      "no-use-extend-native/no-use-extend-native": "off",
+      "unicorn/prevent-abbreviations": "off"
     }
   }
 }