How to use native express middleware in controller or REST decorator? about node-decorators HOT 9 CLOSED

can you explain a little bit more maybe put an example

from node-decorators.

Hi @markkelsall. At the moment simple functions are not supported. All the middlewares have to be classes. In order to use plain function you would need to wrap it. Is it inconvenient to use middleware as classes?

from node-decorators.

Hi @serhiisol ok got it.
Yes ideally it would be better to be able to use simple functions (async included) as middleware as well as classes.
I was under the impression that the code supported it and it was just the typings that I was getting wrong, based on this - https://github.com/serhiisol/node-decorators/blob/master/express/src/middleware.ts

from node-decorators.

I’m going to add support of the simple functions

from node-decorators.

That would be great, will make adoption for new devs a lot easier!
Do let me know when you have a change ready to look at

from node-decorators.

Hi, @serhiisol, is the only change for this issue/feature request the one commit then? I saw there are a few things going in as part of this linked PR and results in a new major version

from node-decorators.

Most of the changes are required and probably it might break something.That's why I called it major changes. For example, if you're returning something you would need to be able to setup status code. Then there were few bugs that I fixed. Technically I can only revert the update of express.

from node-decorators.

Ok, I’ll try to split the changes into a bunch of PRs

from node-decorators.

Ok great, let me know how you’re getting on and whether it’s just that simple change you had previously

from node-decorators.