-
Notifications
You must be signed in to change notification settings - Fork 3
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
Async support? #5
Comments
Hello, Yes, we could support await/async. Have you tried to use the native NodeJS function to promisify rock.req with util.promisify?
Not yet. In general, it is slower. |
Nice, I haven't played around with (or even heard of) Will try with that approach and see if I can get it to work. However, still do agree that it would be great if |
Is there any future plans for async support, e.g. with
await
?I like libraries like
node-fetch
andaxios
, as one of the things they ship "out of the box" is ability to use promise and support async-await pattern in node/express.My question, has the benchmarks for this library - which are pretty impressive btw - been tested in an async environment? Also, how would one go about "promisifying" a library such as
rock-req
, if I may ask? That is, would it be possible to wrap such a library in anasync
block to use withawait
?The text was updated successfully, but these errors were encountered: