Skip to content

Express: Add middlewareFactory(deps)(middleware)#51

Merged
vlaurin merged 2 commits intomasterfrom
express/middleware-factory
Apr 26, 2020
Merged

Express: Add middlewareFactory(deps)(middleware)#51
vlaurin merged 2 commits intomasterfrom
express/middleware-factory

Conversation

@vlaurin
Copy link
Copy Markdown
Contributor

@vlaurin vlaurin commented Apr 26, 2020

Fixes #49

import {middlewareFactory} from '@quickcase/node-toolkit';
import express from 'express';
const router = express.Router();
const dependencySuppliers = {
  tokenProvider: (req) => () => Promise.resolve(req.accessToken),
  service: (req) => {...},
};
const middlewareSupplier = ({tokenProvider, service}) => (req, res) => {
  // use initialised `tokenProvider` and `service`...
};
router.get('/', middlewareFactory(dependencySuppliers)(middlewareSupplier));

@vlaurin vlaurin requested a review from gbenadikar April 26, 2020 21:13
@vlaurin vlaurin merged commit f3a71d6 into master Apr 26, 2020
@vlaurin vlaurin deleted the express/middleware-factory branch April 26, 2020 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Express: middlewareFactory(dynamicDependencies)(middlewareConstructor)

1 participant