Ocelot icon indicating copy to clipboard operation
Ocelot copied to clipboard

Add 'multipart/form-data' support for file uploads

Open myall200 opened this issue 4 years ago • 4 comments

Fixes #714

  • #714

Motivation

Introduce a support of file streaming (file uploads) via the gateway, without body content caching. Current loading of entire body to a memory in Ocelot core is a source of errors, unstable responses because downstream services produce errors during long lasting operation of the request.

Proposed Changes

  • add multipart/form-data support for file uploads

Related to

  • #792
  • #714
  • #617
  • #1685

myall200 avatar Nov 13 '20 08:11 myall200

Hi zenghao! Thanks for your interest in Ocelot!

Why did you revert the changes by reverting commit?

Could you Sync fork please? There is no develop branch in the forked repo!

raman-m avatar Jul 15 '23 14:07 raman-m

Related to:

  • #792
  • #714
  • #617
  • #1685

raman-m avatar Aug 11 '23 13:08 raman-m

@myall200 Hi Zenghao! Do you have any intention of contributing to this feature further?

I hope, if we will have a pair programming sessions together, we could deliver full functioning feature and the PR could close the list of open issues.

Happy Friday! Have a nice sunny weekend in China! 😉

raman-m avatar Aug 11 '23 18:08 raman-m

The feature branch (master) has been rebased onto ThreeMammals:develop!


I don't see develop branch in your fork! Your fork is too old.

Could you Sync fork please? So, new develop branch will occur with all top commits!

Could you add me as collaborator to your forked repo please? I will create develop branch and make it default.


There are 2 failed tests in the build:

Fully qualified names:

Reason: [Incorrect Content-Type:] I guess it won't be hard to fix these failed tests.

raman-m avatar Aug 11 '23 19:08 raman-m