@ladjs/api 8.0.0 → 9.0.0

package/index.js

@@ -5,6 +5,7 @@ const util = require('util');
 
 const Cabin = require('cabin');
 const I18N = require('@ladjs/i18n');
+const Passport = require('@ladjs/passport');
 const Koa = require('koa');
 const Redis = require('@ladjs/redis');
 const StoreIPAddress = require('@ladjs/store-ip-address');
@@ -19,49 +20,65 @@ const etag = require('koa-etag');
 const json = require('koa-json');
 const koa404Handler = require('koa-404-handler');
 const koaConnect = require('koa-connect');
+const ms = require('ms');
 const multimatch = require('multimatch');
+const ratelimit = require('@ladjs/koa-simple-ratelimit');
 const removeTrailingSlashes = require('koa-no-trailing-slash');
 const requestId = require('express-request-id');
 const requestReceived = require('request-received');
 const responseTime = require('response-time');
 const sharedConfig = require('@ladjs/shared-config');
 const { boolean } = require('boolean');
-const { ratelimit } = require('koa-simple-ratelimit');
+
+const RATE_LIMIT_EXCEEDED = `Rate limit exceeded, retry in %s.`;
 
 class API {
-  constructor(config, client) {
+  // eslint-disable-next-line complexity
+  constructor(config, Users) {
     this.config = {
       ...sharedConfig('API'),
       rateLimitIgnoredGlobs: [],
       ...config
     };
 
-    const cabin = new Cabin({
-      logger: this.config.logger,
-      ...this.config.cabin
-    });
-
-    // initialize redis
-    this.client = client
-      ? client
-      : new Redis(this.config.redis, cabin, this.config.redisMonitor);
-
     // initialize the app
     const app = new Koa();
 
-    // allow middleware to access redis client
+    // initialize cabin
+    this.logger = _.isPlainObject(this.config.logger)
+      ? new Cabin(this.config.logger)
+      : this.config.logger instanceof Cabin
+      ? this.config.logger
+      : new Cabin({
+          logger: this.config.logger ? this.config.logger : console
+        });
+    app.context.logger = this.logger;
+
+    // initialize redis
+    this.client =
+      this.config.redis === false
+        ? false
+        : _.isPlainObject(this.config.redis)
+        ? new Redis(this.config.redis, this.logger, this.config.redisMonitor)
+        : this.config.redis;
     app.context.client = this.client;
 
-    // listen for error and log events emitted by app
+    // expose passport
+    this.passport =
+      this.config.passport === false
+        ? false
+        : _.isPlainObject(this.config.passport)
+        ? new Passport(this.config.passport, Users)
+        : this.config.passport;
+    app.context.passport = this.passport;
+
+    // listen for errors emitted by app
     app.on('error', (err, ctx) => {
-      const level = err.status && err.status < 500 ? 'warn' : 'error';
-      if (ctx.logger) ctx.logger[level](err);
-      else cabin[level](err);
+      ctx.logger[err.status && err.status < 500 ? 'warn' : 'error'](err);
     });
-    app.on('log', cabin.log);
 
     // override koa's undocumented error handler
-    app.context.onerror = errorHandler(false, cabin);
+    app.context.onerror = errorHandler(false, this.logger);
 
     // only trust proxy if enabled
     app.proxy = boolean(process.env.TRUST_PROXY);
@@ -86,7 +103,7 @@ class API {
     app.use(koaConnect(requestId()));
 
     // use the cabin middleware (adds request-based logging and helpers)
-    app.use(cabin.middleware);
+    app.use(this.logger.middleware);
 
     // allow before hooks to get setup
     if (_.isFunction(this.config.hookBeforeSetup))
@@ -96,7 +113,7 @@ class API {
     if (this.config.auth) app.use(auth(this.config.auth));
 
     // rate limiting
-    if (this.config.rateLimit) {
+    if (this.client && this.config.rateLimit) {
       app.use((ctx, next) => {
         // check against ignored/whitelisted paths
         if (
@@ -109,7 +126,14 @@ class API {
 
         return ratelimit({
           ...this.config.rateLimit,
-          db: this.client
+          db: this.client,
+          logger: this.logger,
+          errorMessage(exp) {
+            const fn =
+              typeof ctx.request.t === 'function' ? ctx.request.t : util.format;
+            // NOTE: ms does not support i18n localization
+            return fn(RATE_LIMIT_EXCEEDED, ms(exp, { long: true }));
+          }
         })(ctx, next);
       });
     }
@@ -121,7 +145,7 @@ class API {
     if (this.config.i18n) {
       const i18n = this.config.i18n.config
         ? this.config.i18n
-        : new I18N({ ...this.config.i18n, logger: cabin });
+        : new I18N({ ...this.config.i18n, logger: this.logger });
       app.use(i18n.middleware);
     }
 
@@ -141,12 +165,12 @@ class API {
     app.use(json());
 
     // passport
-    if (this.config.passport) app.use(this.config.passport.initialize());
+    if (this.passport) app.use(this.passport.initialize());
 
     // store the user's last ip address in the background
     if (this.config.storeIPAddress) {
       const storeIPAddress = new StoreIPAddress({
-        logger: cabin,
+        logger: this.logger,
         ...this.config.storeIPAddress
       });
       app.use(storeIPAddress.middleware);

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@ladjs/api",
   "description": "API server for Lad",
-  "version": "8.0.0",
+  "version": "9.0.0",
   "author": "Nick Baugh <niftylettuce@gmail.com> (http://niftylettuce.com/)",
   "ava": {
     "failFast": true,
@@ -22,8 +22,9 @@
   "dependencies": {
     "@koa/router": "^10.1.1",
     "@ladjs/i18n": "^7.2.6",
+    "@ladjs/koa-simple-ratelimit": "^3.0.0",
     "@ladjs/redis": "^1.0.7",
-    "@ladjs/shared-config": "^6.0.0",
+    "@ladjs/shared-config": "^7.0.3",
     "@ladjs/store-ip-address": "^0.0.7",
     "boolean": "^3.2.0",
     "cabin": "^9.1.2",
@@ -41,29 +42,30 @@
     "koa-etag": "^4.0.0",
     "koa-json": "^2.0.2",
     "koa-no-trailing-slash": "^2.1.0",
-    "koa-simple-ratelimit": "^5.1.1",
     "lodash": "^4.17.21",
+    "ms": "^2.1.3",
     "multimatch": "5",
     "request-received": "^0.0.3",
     "response-time": "^2.3.2"
   },
   "devDependencies": {
-    "@commitlint/cli": "^16.2.4",
-    "@commitlint/config-conventional": "^16.2.4",
-    "ava": "^4.2.0",
+    "@commitlint/cli": "^17.0.2",
+    "@commitlint/config-conventional": "^17.0.2",
+    "@ladjs/passport": "^5.0.0",
+    "ava": "^4.3.0",
     "codecov": "^3.8.3",
     "cross-env": "^7.0.3",
-    "eslint": "^8.14.0",
+    "eslint": "^8.16.0",
     "eslint-config-xo-lass": "^1.0.6",
     "fixpack": "^4.0.0",
-    "husky": "^7.0.4",
-    "lint-staged": "12.4.1",
-    "mongoose": "^6.3.2",
+    "husky": "^8.0.1",
+    "lint-staged": "13.0.0",
+    "mongoose": "^6.3.5",
     "nyc": "^15.1.0",
     "remark-cli": "^10.0.1",
-    "remark-preset-github": "^4.0.1",
+    "remark-preset-github": "^4.0.2",
     "supertest": "^6.2.3",
-    "xo": "^0.48.0"
+    "xo": "^0.49.0"
   },
   "engines": {
     "node": ">=10.10.0"