Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add support for Restify #118

Closed
slavafomin opened this issue Apr 24, 2017 · 6 comments
Closed

feat: add support for Restify #118

slavafomin opened this issue Apr 24, 2017 · 6 comments
Labels
status: wontfix type: feature Issues related to new features.

Comments

@slavafomin
Copy link
Contributor

restify is a node.js module built specifically to enable you to build correct REST web services. It intentionally borrows heavily from express as that is more or less the de facto API for writing web applications on top of node.js.

Restify has a very similar API to Express, but nevertheless has issues when used with Express driver. Probably, it will need it's own driver for clarity and maintainability.

@pleerock
Copy link
Contributor

Im not against it if you will maintain it

@slavafomin
Copy link
Contributor Author

My resources are very limited at the moment (I think it goes for most of us here), but I definitely would like support for Restify, so I will try to implement it in the future, if no one does this before me of course.

@bilby91
Copy link

bilby91 commented Oct 5, 2017

Can anyone point to the areas that need adaption ? We are interested in using restify too.

@MichalLytek
Copy link
Contributor

MichalLytek commented Oct 5, 2017

I've added support for custom drivers, so all you need to do is extend the BaseDriver class. Then you can create a routing-controllers app with createServer function.

Once you will be satisfied with the restify driver, you can create a PR with the new driver, but you also have to change test suite for the new driver 😉

@github-actions
Copy link

Stale issue message

@github-actions github-actions bot added the status: awaiting answer Awaiting answer from the author of issue or PR. label Feb 18, 2020
@NoNameProvided NoNameProvided added type: feature Issues related to new features. status: wontfix and removed status: awaiting answer Awaiting answer from the author of issue or PR. labels Aug 9, 2020
@NoNameProvided
Copy link
Member

We are moving away from custom/multiple driver support. RC will be shipped with a fixed driver.

@typestack typestack locked as resolved and limited conversation to collaborators Aug 9, 2020
@NoNameProvided NoNameProvided changed the title Add support for Restify feat: add support for Restify Aug 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: wontfix type: feature Issues related to new features.
Development

No branches or pull requests

5 participants