@feathersjs/express 5.0.0-pre.6 → 5.0.0

package/lib/rest.js

@@ -1,38 +1,57 @@
 "use strict";
-var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, generator) {
-    function adopt(value) { return value instanceof P ? value : new P(function (resolve) { resolve(value); }); }
-    return new (P || (P = Promise))(function (resolve, reject) {
-        function fulfilled(value) { try { step(generator.next(value)); } catch (e) { reject(e); } }
-        function rejected(value) { try { step(generator["throw"](value)); } catch (e) { reject(e); } }
-        function step(result) { result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected); }
-        step((generator = generator.apply(thisArg, _arguments || [])).next());
-    });
-};
-var __rest = (this && this.__rest) || function (s, e) {
-    var t = {};
-    for (var p in s) if (Object.prototype.hasOwnProperty.call(s, p) && e.indexOf(p) < 0)
-        t[p] = s[p];
-    if (s != null && typeof Object.getOwnPropertySymbols === "function")
-        for (var i = 0, p = Object.getOwnPropertySymbols(s); i < p.length; i++) {
-            if (e.indexOf(p[i]) < 0 && Object.prototype.propertyIsEnumerable.call(s, p[i]))
-                t[p[i]] = s[p[i]];
-        }
-    return t;
-};
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.rest = exports.serviceMethodHandler = exports.serviceMiddleware = exports.formatter = exports.feathersParams = void 0;
+exports.rest = exports.formatter = void 0;
+const express_1 = require("express");
 const errors_1 = require("@feathersjs/errors");
 const commons_1 = require("@feathersjs/commons");
 const transport_commons_1 = require("@feathersjs/transport-commons");
 const feathers_1 = require("@feathersjs/feathers");
-const express_1 = require("express");
 const authentication_1 = require("./authentication");
-const debug = commons_1.createDebug('@feathersjs/express/rest');
-const feathersParams = (req, _res, next) => {
-    req.feathers = Object.assign(Object.assign({}, req.feathers), { provider: 'rest', headers: req.headers });
-    next();
+const debug = (0, commons_1.createDebug)('@feathersjs/express/rest');
+const toHandler = (func) => {
+    return (req, res, next) => func(req, res, next).catch((error) => next(error));
+};
+const serviceMiddleware = () => {
+    return toHandler(async (req, res, next) => {
+        const { query, headers, path, body: data, method: httpMethod } = req;
+        const methodOverride = req.headers[transport_commons_1.http.METHOD_HEADER];
+        // eslint-disable-next-line @typescript-eslint/no-non-null-assertion
+        const { service, params: { __id: id = null, ...route } = {} } = req.lookup;
+        const method = transport_commons_1.http.getServiceMethod(httpMethod, id, methodOverride);
+        const { methods } = (0, feathers_1.getServiceOptions)(service);
+        debug(`Found service for path ${path}, attempting to run '${method}' service method`);
+        if (!methods.includes(method) || feathers_1.defaultServiceMethods.includes(methodOverride)) {
+            const error = new errors_1.MethodNotAllowed(`Method \`${method}\` is not supported by this endpoint.`);
+            res.statusCode = error.code;
+            throw error;
+        }
+        const createArguments = transport_commons_1.http.argumentsFor[method] || transport_commons_1.http.argumentsFor.default;
+        const params = { query, headers, route, ...req.feathers };
+        const args = createArguments({ id, data, params });
+        const contextBase = (0, feathers_1.createContext)(service, method, { http: {} });
+        res.hook = contextBase;
+        const context = await service[method](...args, contextBase);
+        res.hook = context;
+        const response = transport_commons_1.http.getResponse(context);
+        res.statusCode = response.status;
+        res.set(response.headers);
+        res.data = response.body;
+        return next();
+    });
+};
+const servicesMiddleware = () => {
+    return toHandler(async (req, res, next) => {
+        const app = req.app;
+        const lookup = app.lookup(req.path);
+        if (!lookup) {
+            return next();
+        }
+        req.lookup = lookup;
+        const options = (0, feathers_1.getServiceOptions)(lookup.service);
+        const middleware = options.express.composed;
+        return middleware(req, res, next);
+    });
 };
-exports.feathersParams = feathersParams;
 const formatter = (_req, res, next) => {
     if (res.data === undefined) {
         return next();
@@ -44,73 +63,24 @@ const formatter = (_req, res, next) => {
     });
 };
 exports.formatter = formatter;
-const serviceMiddleware = (callback) => (req, res, next) => __awaiter(void 0, void 0, void 0, function* () {
-    debug(`Running service middleware for '${req.url}'`);
-    try {
-        const { query, body: data } = req;
-        const _a = req.params, { __feathersId: id = null } = _a, route = __rest(_a, ["__feathersId"]);
-        const params = Object.assign({ query, route }, req.feathers);
-        const context = yield callback(req, res, { id, data, params });
-        const result = transport_commons_1.http.getData(context);
-        res.data = result;
-        res.status(transport_commons_1.http.getStatusCode(context, result));
-        next();
-    }
-    catch (error) {
-        next(error);
-    }
-});
-exports.serviceMiddleware = serviceMiddleware;
-const serviceMethodHandler = (service, methodName, getArgs, headerOverride) => exports.serviceMiddleware((req, res, options) => __awaiter(void 0, void 0, void 0, function* () {
-    const methodOverride = typeof headerOverride === 'string' && req.headers[headerOverride];
-    const method = methodOverride ? methodOverride : methodName;
-    const { methods } = feathers_1.getServiceOptions(service);
-    if (!methods.includes(method) || feathers_1.defaultServiceMethods.includes(methodOverride)) {
-        res.status(transport_commons_1.http.statusCodes.methodNotAllowed);
-        throw new errors_1.MethodNotAllowed(`Method \`${method}\` is not supported by this endpoint.`);
-    }
-    const args = getArgs(options);
-    const context = feathers_1.createContext(service, method);
-    res.hook = context;
-    return service[method](...args, context);
-}));
-exports.serviceMethodHandler = serviceMethodHandler;
-function rest(handler = exports.formatter) {
-    return function (app) {
+const rest = (options) => {
+    options = typeof options === 'function' ? { formatter: options } : options || {};
+    const formatterMiddleware = options.formatter || exports.formatter;
+    const authenticationOptions = options.authentication;
+    return (app) => {
         if (typeof app.route !== 'function') {
             throw new Error('@feathersjs/express/rest needs an Express compatible app.');
         }
-        app.use(exports.feathersParams);
-        app.use(authentication_1.parseAuthentication());
-        // Register the REST provider
-        app.mixins.push(function (service, path, options) {
-            const { middleware: { before = [] } } = options;
-            let { middleware: { after = [] } } = options;
-            if (typeof handler === 'function') {
-                after = after.concat(handler);
-            }
-            const baseUri = `/${path}`;
-            const find = exports.serviceMethodHandler(service, 'find', transport_commons_1.http.argumentsFor.find);
-            const get = exports.serviceMethodHandler(service, 'get', transport_commons_1.http.argumentsFor.get);
-            const create = exports.serviceMethodHandler(service, 'create', transport_commons_1.http.argumentsFor.create, transport_commons_1.http.METHOD_HEADER);
-            const update = exports.serviceMethodHandler(service, 'update', transport_commons_1.http.argumentsFor.update);
-            const patch = exports.serviceMethodHandler(service, 'patch', transport_commons_1.http.argumentsFor.patch);
-            const remove = exports.serviceMethodHandler(service, 'remove', transport_commons_1.http.argumentsFor.remove);
-            debug(`Adding REST provider for service \`${path}\` at base route \`${baseUri}\``);
-            const idRoute = '/:__feathersId';
-            const serviceRouter = express_1.Router({ mergeParams: true })
-                .get('/', find)
-                .post('/', create)
-                .get(idRoute, get)
-                .put('/', update)
-                .put(idRoute, update)
-                .patch('/', patch)
-                .patch(idRoute, patch)
-                .delete('/', remove)
-                .delete(idRoute, remove);
-            app.use(baseUri, ...before, serviceRouter, ...after);
+        app.use((0, authentication_1.parseAuthentication)(authenticationOptions));
+        app.use(servicesMiddleware());
+        app.mixins.push((_service, _path, options) => {
+            const { express: { before = [], after = [] } = {} } = options;
+            const middlewares = [].concat(before, serviceMiddleware(), after, formatterMiddleware);
+            const middleware = (0, express_1.Router)().use(middlewares);
+            options.express || (options.express = {});
+            options.express.composed = middleware;
         });
     };
-}
+};
 exports.rest = rest;
 //# sourceMappingURL=rest.js.map

package/lib/rest.js.map

@@ -1 +1 @@
-{"version":3,"file":"rest.js","sourceRoot":"","sources":["../src/rest.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;AAAA,+CAAsD;AAEtD,iDAAkD;AAClD,qEAAqD;AACrD,mDAA+F;AAC/F,qCAAkF;AAElF,qDAAuD;AAEvD,MAAM,KAAK,GAAG,qBAAW,CAAC,0BAA0B,CAAC,CAAC;AAI/C,MAAM,cAAc,GAAG,CAAC,GAAY,EAAE,IAAc,EAAE,IAAkB,EAAE,EAAE;IACjF,GAAG,CAAC,QAAQ,mCACP,GAAG,CAAC,QAAQ,KACf,QAAQ,EAAE,MAAM,EAChB,OAAO,EAAE,GAAG,CAAC,OAAO,GACrB,CAAC;IACF,IAAI,EAAE,CAAC;AACT,CAAC,CAAA;AAPY,QAAA,cAAc,kBAO1B;AAEM,MAAM,SAAS,GAAG,CAAC,IAAa,EAAE,GAAa,EAAE,IAAkB,EAAE,EAAE;IAC5E,IAAI,GAAG,CAAC,IAAI,KAAK,SAAS,EAAE;QAC1B,OAAO,IAAI,EAAE,CAAC;KACf;IAED,GAAG,CAAC,MAAM,CAAC;QACT,kBAAkB;YAChB,GAAG,CAAC,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;QACrB,CAAC;KACF,CAAC,CAAC;AACL,CAAC,CAAA;AAVY,QAAA,SAAS,aAUrB;AAGM,MAAM,iBAAiB,GAAG,CAAC,QAAyB,EAAE,EAAE,CAC7D,CAAO,GAAY,EAAE,GAAa,EAAE,IAAkB,EAAE,EAAE;IACxD,KAAK,CAAC,mCAAmC,GAAG,CAAC,GAAG,GAAG,CAAC,CAAC;IAErD,IAAI;QACF,MAAM,EAAE,KAAK,EAAE,IAAI,EAAE,IAAI,EAAE,GAAG,GAAG,CAAC;QAClC,MAAM,KAAwC,GAAG,CAAC,MAAM,EAAlD,EAAE,YAAY,EAAE,EAAE,GAAG,IAAI,OAAyB,EAApB,KAAK,cAAnC,gBAAqC,CAAa,CAAC;QACzD,MAAM,MAAM,mBAAK,KAAK,EAAE,KAAK,IAAK,GAAG,CAAC,QAAQ,CAAE,CAAC;QACjD,MAAM,OAAO,GAAG,MAAM,QAAQ,CAAC,GAAG,EAAE,GAAG,EAAE,EAAE,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC,CAAC;QAC/D,MAAM,MAAM,GAAG,wBAAI,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC;QAErC,GAAG,CAAC,IAAI,GAAG,MAAM,CAAC;QAClB,GAAG,CAAC,MAAM,CAAC,wBAAI,CAAC,aAAa,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,CAAC;QAEhD,IAAI,EAAE,CAAC;KACR;IAAC,OAAO,KAAK,EAAE;QACd,IAAI,CAAC,KAAK,CAAC,CAAC;KACb;AACH,CAAC,CAAA,CAAA;AAlBU,QAAA,iBAAiB,qBAkB3B;AAEI,MAAM,oBAAoB,GAAG,CAClC,OAAY,EAAE,UAAkB,EAAE,OAA4C,EAAE,cAAuB,EACvG,EAAE,CAAC,yBAAiB,CAAC,CAAO,GAAG,EAAE,GAAG,EAAE,OAAO,EAAE,EAAE;IACjD,MAAM,cAAc,GAAG,OAAO,cAAc,KAAK,QAAQ,IAAK,GAAG,CAAC,OAAO,CAAC,cAAc,CAAY,CAAC;IACrG,MAAM,MAAM,GAAG,cAAc,CAAC,CAAC,CAAC,cAAc,CAAC,CAAC,CAAC,UAAU,CAAA;IAC3D,MAAM,EAAE,OAAO,EAAE,GAAG,4BAAiB,CAAC,OAAO,CAAC,CAAC;IAE/C,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAC,IAAI,gCAAqB,CAAC,QAAQ,CAAC,cAAc,CAAC,EAAE;QAC/E,GAAG,CAAC,MAAM,CAAC,wBAAI,CAAC,WAAW,CAAC,gBAAgB,CAAC,CAAC;QAE9C,MAAM,IAAI,yBAAgB,CAAC,YAAY,MAAM,uCAAuC,CAAC,CAAC;KACvF;IAED,MAAM,IAAI,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC;IAC9B,MAAM,OAAO,GAAG,wBAAa,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;IAE/C,GAAG,CAAC,IAAI,GAAG,OAAc,CAAC;IAE1B,OAAO,OAAO,CAAC,MAAM,CAAC,CAAC,GAAG,IAAI,EAAE,OAAO,CAAC,CAAC;AAC3C,CAAC,CAAA,CAAC,CAAC;AAnBU,QAAA,oBAAoB,wBAmB9B;AAEH,SAAgB,IAAI,CAAE,UAA0B,iBAAS;IACvD,OAAO,UAAqB,GAAQ;QAClC,IAAI,OAAO,GAAG,CAAC,KAAK,KAAK,UAAU,EAAE;YACnC,MAAM,IAAI,KAAK,CAAC,2DAA2D,CAAC,CAAC;SAC9E;QAED,GAAG,CAAC,GAAG,CAAC,sBAAc,CAAC,CAAC;QACxB,GAAG,CAAC,GAAG,CAAC,oCAAmB,EAAE,CAAC,CAAC;QAE/B,6BAA6B;QAC7B,GAAG,CAAC,MAAM,CAAC,IAAI,CAAC,UAAU,OAAY,EAAE,IAAY,EAAE,OAAY;YAChE,MAAM,EAAE,UAAU,EAAE,EAAE,MAAM,GAAG,EAAE,EAAE,EAAE,GAAG,OAAO,CAAC;YAChD,IAAI,EAAE,UAAU,EAAE,EAAE,KAAK,GAAG,EAAE,EAAE,EAAE,GAAG,OAAO,CAAC;YAE7C,IAAI,OAAO,OAAO,KAAK,UAAU,EAAE;gBACjC,KAAK,GAAG,KAAK,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;aAC/B;YAED,MAAM,OAAO,GAAG,IAAI,IAAI,EAAE,CAAC;YAC3B,MAAM,IAAI,GAAG,4BAAoB,CAAC,OAAO,EAAE,MAAM,EAAE,wBAAI,CAAC,YAAY,CAAC,IAAI,CAAC,CAAC;YAC3E,MAAM,GAAG,GAAG,4BAAoB,CAAC,OAAO,EAAE,KAAK,EAAE,wBAAI,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC;YACxE,MAAM,MAAM,GAAG,4BAAoB,CAAC,OAAO,EAAE,QAAQ,EAAE,wBAAI,CAAC,YAAY,CAAC,MAAM,EAAE,wBAAI,CAAC,aAAa,CAAC,CAAC;YACrG,MAAM,MAAM,GAAG,4BAAoB,CAAC,OAAO,EAAE,QAAQ,EAAE,wBAAI,CAAC,YAAY,CAAC,MAAM,CAAC,CAAC;YACjF,MAAM,KAAK,GAAG,4BAAoB,CAAC,OAAO,EAAE,OAAO,EAAE,wBAAI,CAAC,YAAY,CAAC,KAAK,CAAC,CAAC;YAC9E,MAAM,MAAM,GAAG,4BAAoB,CAAC,OAAO,EAAE,QAAQ,EAAE,wBAAI,CAAC,YAAY,CAAC,MAAM,CAAC,CAAC;YAEjF,KAAK,CAAC,sCAAsC,IAAI,sBAAsB,OAAO,IAAI,CAAC,CAAC;YAEnF,MAAM,OAAO,GAAG,gBAAgB,CAAC;YACjC,MAAM,aAAa,GAAG,gBAAM,CAAC,EAAE,WAAW,EAAE,IAAI,EAAE,CAAC;iBAChD,GAAG,CAAC,GAAG,EAAE,IAAI,CAAC;iBACd,IAAI,CAAC,GAAG,EAAE,MAAM,CAAC;iBACjB,GAAG,CAAC,OAAO,EAAE,GAAG,CAAC;iBACjB,GAAG,CAAC,GAAG,EAAE,MAAM,CAAC;iBAChB,GAAG,CAAC,OAAO,EAAE,MAAM,CAAC;iBACpB,KAAK,CAAC,GAAG,EAAE,KAAK,CAAC;iBACjB,KAAK,CAAC,OAAO,EAAE,KAAK,CAAC;iBACrB,MAAM,CAAC,GAAG,EAAE,MAAM,CAAC;iBACnB,MAAM,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;YAE3B,GAAG,CAAC,GAAG,CAAC,OAAO,EAAE,GAAG,MAAM,EAAE,aAAa,EAAE,GAAG,KAAK,CAAC,CAAC;QACvD,CAAC,CAAC,CAAC;IACL,CAAC,CAAC;AACJ,CAAC;AA3CD,oBA2CC"}
+{"version":3,"file":"rest.js","sourceRoot":"","sources":["../src/rest.ts"],"names":[],"mappings":";;;AAAA,qCAAmE;AACnE,+CAAqD;AACrD,iDAAiD;AACjD,qEAAoD;AACpD,mDAA8F;AAE9F,qDAA8E;AAG9E,MAAM,KAAK,GAAG,IAAA,qBAAW,EAAC,0BAA0B,CAAC,CAAA;AAErD,MAAM,SAAS,GAAG,CAChB,IAAsE,EACtD,EAAE;IAClB,OAAO,CAAC,GAAG,EAAE,GAAG,EAAE,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,EAAE,IAAI,CAAC,CAAC,KAAK,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAA;AAC/E,CAAC,CAAA;AAED,MAAM,iBAAiB,GAAG,GAAmB,EAAE;IAC7C,OAAO,SAAS,CAAC,KAAK,EAAE,GAAG,EAAE,GAAG,EAAE,IAAI,EAAE,EAAE;QACxC,MAAM,EAAE,KAAK,EAAE,OAAO,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,EAAE,MAAM,EAAE,UAAU,EAAE,GAAG,GAAG,CAAA;QACpE,MAAM,cAAc,GAAG,GAAG,CAAC,OAAO,CAAC,wBAAI,CAAC,aAAa,CAAuB,CAAA;QAE5E,oEAAoE;QACpE,MAAM,EAAE,OAAO,EAAE,MAAM,EAAE,EAAE,IAAI,EAAE,EAAE,GAAG,IAAI,EAAE,GAAG,KAAK,EAAE,GAAG,EAAE,EAAE,GAAG,GAAG,CAAC,MAAO,CAAA;QAC3E,MAAM,MAAM,GAAG,wBAAI,CAAC,gBAAgB,CAAC,UAAU,EAAE,EAAE,EAAE,cAAc,CAAC,CAAA;QACpE,MAAM,EAAE,OAAO,EAAE,GAAG,IAAA,4BAAiB,EAAC,OAAO,CAAC,CAAA;QAE9C,KAAK,CAAC,0BAA0B,IAAI,wBAAwB,MAAM,kBAAkB,CAAC,CAAA;QAErF,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAC,IAAI,gCAAqB,CAAC,QAAQ,CAAC,cAAc,CAAC,EAAE;YAC/E,MAAM,KAAK,GAAG,IAAI,yBAAgB,CAAC,YAAY,MAAM,uCAAuC,CAAC,CAAA;YAC7F,GAAG,CAAC,UAAU,GAAG,KAAK,CAAC,IAAI,CAAA;YAC3B,MAAM,KAAK,CAAA;SACZ;QAED,MAAM,eAAe,GAAG,wBAAI,CAAC,YAAY,CAAC,MAAe,CAAC,IAAI,wBAAI,CAAC,YAAY,CAAC,OAAO,CAAA;QACvF,MAAM,MAAM,GAAG,EAAE,KAAK,EAAE,OAAO,EAAE,KAAK,EAAE,GAAG,GAAG,CAAC,QAAQ,EAAE,CAAA;QACzD,MAAM,IAAI,GAAG,eAAe,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC,CAAA;QAClD,MAAM,WAAW,GAAG,IAAA,wBAAa,EAAC,OAAO,EAAE,MAAM,EAAE,EAAE,IAAI,EAAE,EAAE,EAAE,CAAC,CAAA;QAChE,GAAG,CAAC,IAAI,GAAG,WAAW,CAAA;QAEtB,MAAM,OAAO,GAAG,MAAO,OAAe,CAAC,MAAM,CAAC,CAAC,GAAG,IAAI,EAAE,WAAW,CAAC,CAAA;QACpE,GAAG,CAAC,IAAI,GAAG,OAAO,CAAA;QAElB,MAAM,QAAQ,GAAG,wBAAI,CAAC,WAAW,CAAC,OAAO,CAAC,CAAA;QAC1C,GAAG,CAAC,UAAU,GAAG,QAAQ,CAAC,MAAM,CAAA;QAChC,GAAG,CAAC,GAAG,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAA;QACzB,GAAG,CAAC,IAAI,GAAG,QAAQ,CAAC,IAAI,CAAA;QAExB,OAAO,IAAI,EAAE,CAAA;IACf,CAAC,CAAC,CAAA;AACJ,CAAC,CAAA;AAED,MAAM,kBAAkB,GAAG,GAAmB,EAAE;IAC9C,OAAO,SAAS,CAAC,KAAK,EAAE,GAAG,EAAE,GAAG,EAAE,IAAI,EAAE,EAAE;QACxC,MAAM,GAAG,GAAG,GAAG,CAAC,GAAyB,CAAA;QACzC,MAAM,MAAM,GAAG,GAAG,CAAC,MAAM,CAAC,GAAG,CAAC,IAAI,CAAC,CAAA;QAEnC,IAAI,CAAC,MAAM,EAAE;YACX,OAAO,IAAI,EAAE,CAAA;SACd;QAED,GAAG,CAAC,MAAM,GAAG,MAAM,CAAA;QAEnB,MAAM,OAAO,GAAG,IAAA,4BAAiB,EAAC,MAAM,CAAC,OAAO,CAAC,CAAA;QACjD,MAAM,UAAU,GAAG,OAAO,CAAC,OAAO,CAAC,QAAQ,CAAA;QAE3C,OAAO,UAAU,CAAC,GAAG,EAAE,GAAG,EAAE,IAAI,CAAC,CAAA;IACnC,CAAC,CAAC,CAAA;AACJ,CAAC,CAAA;AAEM,MAAM,SAAS,GAAmB,CAAC,IAAI,EAAE,GAAG,EAAE,IAAI,EAAE,EAAE;IAC3D,IAAI,GAAG,CAAC,IAAI,KAAK,SAAS,EAAE;QAC1B,OAAO,IAAI,EAAE,CAAA;KACd;IAED,GAAG,CAAC,MAAM,CAAC;QACT,kBAAkB;YAChB,GAAG,CAAC,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,CAAA;QACpB,CAAC;KACF,CAAC,CAAA;AACJ,CAAC,CAAA;AAVY,QAAA,SAAS,aAUrB;AAOM,MAAM,IAAI,GAAG,CAAC,OAAsC,EAAE,EAAE;IAC7D,OAAO,GAAG,OAAO,OAAO,KAAK,UAAU,CAAC,CAAC,CAAC,EAAE,SAAS,EAAE,OAAO,EAAE,CAAC,CAAC,CAAC,OAAO,IAAI,EAAE,CAAA;IAEhF,MAAM,mBAAmB,GAAG,OAAO,CAAC,SAAS,IAAI,iBAAS,CAAA;IAC1D,MAAM,qBAAqB,GAAG,OAAO,CAAC,cAAc,CAAA;IAEpD,OAAO,CAAC,GAAgB,EAAE,EAAE;QAC1B,IAAI,OAAO,GAAG,CAAC,KAAK,KAAK,UAAU,EAAE;YACnC,MAAM,IAAI,KAAK,CAAC,2DAA2D,CAAC,CAAA;SAC7E;QAED,GAAG,CAAC,GAAG,CAAC,IAAA,oCAAmB,EAAC,qBAAqB,CAAC,CAAC,CAAA;QACnD,GAAG,CAAC,GAAG,CAAC,kBAAkB,EAAE,CAAC,CAAA;QAE7B,GAAG,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,QAAQ,EAAE,KAAK,EAAE,OAAO,EAAE,EAAE;YAC3C,MAAM,EAAE,OAAO,EAAE,EAAE,MAAM,GAAG,EAAE,EAAE,KAAK,GAAG,EAAE,EAAE,GAAG,EAAE,EAAE,GAAG,OAAO,CAAA;YAE7D,MAAM,WAAW,GAAG,EAAE,CAAC,MAAM,CAAC,MAAM,EAAE,iBAAiB,EAAE,EAAE,KAAK,EAAE,mBAAmB,CAAC,CAAA;YACtF,MAAM,UAAU,GAAG,IAAA,gBAAM,GAAE,CAAC,GAAG,CAAC,WAAW,CAAC,CAAA;YAE5C,OAAO,CAAC,OAAO,KAAf,OAAO,CAAC,OAAO,GAAK,EAAE,EAAA;YACtB,OAAO,CAAC,OAAO,CAAC,QAAQ,GAAG,UAAU,CAAA;QACvC,CAAC,CAAC,CAAA;IACJ,CAAC,CAAA;AACH,CAAC,CAAA;AAxBY,QAAA,IAAI,QAwBhB"}

package/package.json

@@ -1,9 +1,10 @@
 {
   "name": "@feathersjs/express",
   "description": "Feathers Express framework bindings and REST provider",
-  "version": "5.0.0-pre.6",
+  "version": "5.0.0",
   "homepage": "https://feathersjs.com",
   "main": "lib/",
+  "types": "lib/",
   "keywords": [
     "feathers",
     "feathers-plugin"
@@ -15,7 +16,8 @@
   },
   "repository": {
     "type": "git",
-    "url": "git://github.com/feathersjs/feathers.git"
+    "url": "git://github.com/feathersjs/feathers.git",
+    "directory": "packages/express"
   },
   "author": {
     "name": "Feathers contributors",
@@ -39,7 +41,8 @@
   ],
   "scripts": {
     "prepublish": "npm run compile",
-    "compile": "shx rm -rf lib/ && tsc",
+    "pack": "npm pack --pack-destination ../generators/test/build",
+    "compile": "shx rm -rf lib/ && tsc && npm run pack",
     "test": "mocha --config ../../.mocharc.json --recursive test/**.test.ts test/**/*.test.ts"
   },
   "directories": {
@@ -49,26 +52,30 @@
     "access": "public"
   },
   "dependencies": {
-    "@feathersjs/commons": "^5.0.0-pre.6",
-    "@feathersjs/errors": "^5.0.0-pre.6",
-    "@feathersjs/transport-commons": "^5.0.0-pre.6",
-    "@types/express": "^4.17.13",
-    "express": "^4.17.1",
-    "lodash": "^4.17.21"
+    "@feathersjs/authentication": "^5.0.0",
+    "@feathersjs/commons": "^5.0.0",
+    "@feathersjs/errors": "^5.0.0",
+    "@feathersjs/feathers": "^5.0.0",
+    "@feathersjs/transport-commons": "^5.0.0",
+    "@types/compression": "^1.7.2",
+    "@types/express": "^4.17.17",
+    "@types/express-serve-static-core": "^4.17.33",
+    "compression": "^1.7.4",
+    "cors": "^2.8.5",
+    "express": "^4.18.2"
   },
   "devDependencies": {
-    "@feathersjs/authentication": "^5.0.0-pre.6",
-    "@feathersjs/authentication-local": "^5.0.0-pre.6",
-    "@feathersjs/feathers": "^5.0.0-pre.6",
-    "@feathersjs/tests": "^5.0.0-pre.6",
-    "@types/mocha": "^9.0.0",
-    "@types/node": "^16.4.13",
-    "axios": "^0.21.1",
+    "@feathersjs/authentication-local": "^5.0.0",
+    "@feathersjs/tests": "^5.0.0",
+    "@types/lodash": "^4.14.191",
+    "@types/mocha": "^10.0.1",
+    "@types/node": "^18.14.1",
+    "axios": "^1.3.4",
     "lodash": "^4.17.21",
-    "mocha": "^9.0.3",
-    "shx": "^0.3.3",
-    "ts-node": "^10.1.0",
-    "typescript": "^4.3.5"
+    "mocha": "^10.2.0",
+    "shx": "^0.3.4",
+    "ts-node": "^10.9.1",
+    "typescript": "^4.9.5"
   },
-  "gitHead": "738f84f91fb72d5e10a0bf57b58db75b0b03d3ff"
+  "gitHead": "90caf635aec850550b9d37bea2762af959d9e8d5"
 }

package/src/authentication.ts

@@ -1,71 +1,66 @@
-import { createDebug } from '@feathersjs/commons';
-import { merge, flatten } from 'lodash';
-import { NextFunction, RequestHandler } from 'express';
+import { RequestHandler, Request, Response } from 'express'
+import { HookContext } from '@feathersjs/feathers'
+import { createDebug } from '@feathersjs/commons'
+import { authenticate as AuthenticateHook } from '@feathersjs/authentication'
 
-const debug = createDebug('@feathersjs/express/authentication');
+import { Application } from './declarations'
 
-type StrategyOptions = {
-  service?: string;
-  strategies: string[]
-};
+const debug = createDebug('@feathersjs/express/authentication')
 
-const normalizeStrategy = (_settings: string|StrategyOptions, ..._strategies: string[]) =>
-  typeof _settings === 'string'
-    ? { strategies: flatten([ _settings, ..._strategies ]) }
-    : _settings;
+const toHandler = (
+  func: (req: Request, res: Response, next: () => void) => Promise<void>
+): RequestHandler => {
+  return (req, res, next) => func(req, res, next).catch((error) => next(error))
+}
+
+export type AuthenticationSettings = {
+  service?: string
+  strategies?: string[]
+}
 
-export function parseAuthentication (settings: any = {}): RequestHandler {
-  return function (req, res, next) {
-    const app = req.app as any;
-    const service = app.defaultAuthentication ? app.defaultAuthentication(settings.service) : null;
+export function parseAuthentication(settings: AuthenticationSettings = {}): RequestHandler {
+  return toHandler(async (req, res, next) => {
+    const app = req.app as any as Application
+    const service = app.defaultAuthentication?.(settings.service)
 
-    if (service === null) {
-      return next();
+    if (!service) {
+      return next()
     }
 
-    const config = service.configuration;
-    const authStrategies = config.parseStrategies || config.authStrategies || [];
+    const config = service.configuration
+    const authStrategies = settings.strategies || config.parseStrategies || config.authStrategies || []
 
     if (authStrategies.length === 0) {
-      debug('No `authStrategies` or `parseStrategies` found in authentication configuration');
-      return next();
+      debug('No `authStrategies` or `parseStrategies` found in authentication configuration')
+      return next()
     }
 
-    service.parse(req, res, ...authStrategies)
-      .then((authentication: any) => {
-        if (authentication) {
-          debug('Parsed authentication from HTTP header', authentication);
-          merge(req, {
-            authentication,
-            feathers: { authentication }
-          });
-        }
-
-        next();
-      }).catch(next);
-  };
-}
+    const authentication = await service.parse(req, res, ...authStrategies)
 
-export function authenticate (_settings: string|StrategyOptions, ..._strategies: string[]) {
-  const settings = normalizeStrategy(_settings, ..._strategies);
+    if (authentication) {
+      debug('Parsed authentication from HTTP header', authentication)
+      req.feathers = { ...req.feathers, authentication }
+    }
+
+    return next()
+  })
+}
 
-  if (!Array.isArray(settings.strategies) || settings.strategies.length === 0) {
-    throw new Error('\'authenticate\' middleware requires at least one strategy name');
-  }
+export function authenticate(
+  settings: string | AuthenticationSettings,
+  ...strategies: string[]
+): RequestHandler {
+  const hook = AuthenticateHook(settings, ...strategies)
 
-  return (_req: Request, _res: Response, next: NextFunction) => {
-    const req = _req as any;
-    const { app, authentication } = req;
-    const service = app.defaultAuthentication(settings.service);
+  return toHandler(async (req, _res, next) => {
+    const app = req.app as any as Application
+    const params = req.feathers
+    const context = { app, params } as any as HookContext
 
-    debug('Authenticating with Express middleware and strategies', settings.strategies);
+    await hook(context)
 
-    service.authenticate(authentication, req.feathers, ...settings.strategies)
-      .then((authResult: any) => {
-        debug('Merging request with', authResult);
-        merge(req, authResult);
+    req.feathers = context.params
 
-        next();
-      }).catch(next);
-  };
+    return next()
+  })
 }

package/src/declarations.ts

@@ -1,60 +1,68 @@
-import http from 'http';
-import express, { Express } from 'express';
+import http from 'http'
+import express, { Express } from 'express'
 import {
-  Application as FeathersApplication, Params as FeathersParams,
-  HookContext, ServiceMethods, ServiceInterface
-} from '@feathersjs/feathers';
+  Application as FeathersApplication,
+  Params as FeathersParams,
+  HookContext,
+  ServiceMethods,
+  ServiceInterface,
+  RouteLookup
+} from '@feathersjs/feathers'
 
-interface ExpressUseHandler<T, ServiceTypes> {
-  <L extends keyof ServiceTypes & string> (
+interface ExpressUseHandler<T, Services> {
+  <L extends keyof Services & string>(
     path: L,
     ...middlewareOrService: (
-      Express|express.RequestHandler|
-      (keyof any extends keyof ServiceTypes ? ServiceInterface<any> : ServiceTypes[L])
+      | Express
+      | express.RequestHandler
+      | express.RequestHandler[]
+      | (keyof any extends keyof Services ? ServiceInterface : Services[L])
     )[]
-  ): T;
-  (path: string|RegExp, ...expressHandlers: express.RequestHandler[]): T;
-  (...expressHandlers: express.RequestHandler[]): T;
-  (handler: Express|express.ErrorRequestHandler): T;
+  ): T
+  (path: string | RegExp, ...expressHandlers: express.RequestHandler[]): T
+  (...expressHandlers: express.RequestHandler[]): T
+  (handler: Express | express.ErrorRequestHandler): T
 }
 
-export interface ExpressOverrides<ServiceTypes> {
-  listen(port: number, hostname: string, backlog: number, callback?: () => void): Promise<http.Server>;
-  listen(port: number, hostname: string, callback?: () => void): Promise<http.Server>;
-  listen(port: number|string|any, callback?: () => void): Promise<http.Server>;
-  listen(callback?: () => void): Promise<http.Server>;
-  use: ExpressUseHandler<this, ServiceTypes>;
+export interface ExpressOverrides<Services> {
+  listen(port: number, hostname: string, backlog: number, callback?: () => void): Promise<http.Server>
+  listen(port: number, hostname: string, callback?: () => void): Promise<http.Server>
+  listen(port: number | string | any, callback?: () => void): Promise<http.Server>
+  listen(callback?: () => void): Promise<http.Server>
+  use: ExpressUseHandler<this, Services>
+  server?: http.Server
 }
 
-export type Application<ServiceTypes = any, AppSettings = any> =
-  Omit<Express, 'listen'|'use'> &
-  FeathersApplication<ServiceTypes, AppSettings> &
-  ExpressOverrides<ServiceTypes>;
+export type Application<Services = any, Settings = any> = Omit<Express, 'listen' | 'use' | 'get' | 'set'> &
+  FeathersApplication<Services, Settings> &
+  ExpressOverrides<Services>
 
 declare module '@feathersjs/feathers/lib/declarations' {
-  export interface ServiceOptions {
-    middleware?: {
-      before: express.RequestHandler[],
-      after: express.RequestHandler[]
+  interface ServiceOptions {
+    express?: {
+      before?: express.RequestHandler[]
+      after?: express.RequestHandler[]
+      composed?: express.RequestHandler
     }
   }
 }
 
 declare module 'express-serve-static-core' {
   interface Request {
-      feathers?: Partial<FeathersParams>;
+    feathers: Partial<FeathersParams> & { [key: string]: any }
+    lookup?: RouteLookup
   }
 
   interface Response {
-      data?: any;
-      hook?: HookContext;
+    data?: any
+    hook?: HookContext
   }
 
   interface IRouterMatcher<T> {
-      // eslint-disable-next-line
-      <P extends Params = ParamsDictionary, ResBody = any, ReqBody = any>(
-          path: PathParams,
-          ...handlers: (RequestHandler<P, ResBody, ReqBody> | Partial<ServiceMethods<any, any>> | Application)[]
-      ): T;
+    // eslint-disable-next-line
+    <P extends Params = ParamsDictionary, ResBody = any, ReqBody = any>(
+      path: PathParams,
+      ...handlers: (RequestHandler<P, ResBody, ReqBody> | Partial<ServiceMethods> | Application)[]
+    ): T
   }
 }