-
Notifications
You must be signed in to change notification settings - Fork 26
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
Expose the processed body/files #15
base: master
Are you sure you want to change the base?
Conversation
This change exposes the resulting request body and/or files value to the outer Koa wrapper for each respective object.
Changed the engine requirements to coordinate with Koa 2.x's requirements of Node 6 or newer.
The tests pass on node 6.x and newer (as per what Koa 2.x requires). |
Changed node versions that are tested since Koa 2.x only requires Node 6 or greater.
can be use |
@techborn thanks |
Will this PR be merged soon? |
@techborn Dont add |
@fundon I can fix this and get it published if you give me collab |
@@ -39,7 +39,10 @@ function makePromise(multer, name) { | |||
if ('request' in ctx){ | |||
if (ctx.req.body) ctx.request.body = ctx.req.body; | |||
if (ctx.req.file) ctx.request.file = ctx.req.file; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should also be ctx.file
.
Please see my comment at #30 (comment) and close this issue. The official Koa organization has forked this repository and released it (with PR's merged) under |
This change exposes the resulting request body and/or files value to the outer Koa wrapper for each respective object.