Open furanzujin opened 2 years ago
Hmm, it looks parser's conflict is solved by time.
Yes, please create a PR with the fixes. Thanks in advance!
As good as done.
Though we made a few more tests to double-check the behavior. It appears that we cannot receive the content-disposition headers provided in each multipart file section. Is there any way to retrieve this information?
I have no idea. You can check the busboy source code.
Hello @icebob,
In the File upload aliases section of the documentation, a note indicates that:
As we need to use both multipart and the JSON body parser on the same route, we experimented a bit with this use case on an auto-aliased route.
Moleculer web routes settings
Service definition
Using the above configurations, it appeared that the three types of routes (standard, multipart, and stream) work together on the same route despite the documentation note.
It is quite good news for us but is this behavior expected to last in further versions? In this case, shall we create a pull request to fix the documentation?
NOTE: also the documentation (Multipart parameters) indicates the
ctx.params.$params
contains parameters from URL querystring, though it appears they are inctx.meta.$params
instead.