flask-restx icon indicating copy to clipboard operation
flask-restx copied to clipboard

Published 20 hours ago • verified publisher icon python-restx

Fix multiform bug

Open jingapore opened this issue 3 years ago • 6 comments

This PR fixes the bug where Swagger does not interpreting an API consuming an array of files, as requiring the request to be a multipart-form. Others have reflected this problem, e.g. #177.

To fix this, this PR makes the following changes over 2 commits.

  • First, expand test-suite to account for an array of files. If this test-suite had been expanded, the original codebase would have failed the test.
  • Second, edit swagger.py to fix the bug. This passes the expanded test suite above.

jingapore avatar Jan 02 '22 16:01 jingapore

@SteadBytes @a-luna @j5awry @ziirish can trouble one of you to look thru this PR? Thanks!

jingapore avatar Jan 17 '22 05:01 jingapore

Was wondering if someone could approve this merge request so that uploads through restx work? Been having a tough time

mdylan2 avatar May 02 '22 21:05 mdylan2

Looks like this got approved and pushed somewhere, but ultimately never got merged?

pbaylies avatar Apr 10 '23 18:04 pbaylies

Does anyone know if this fix will be merged?

gerardoemr avatar Apr 14 '23 12:04 gerardoemr

@gerardoemr @pbaylies Thanks for bumping this, I will try and get it merged over the weekend and released if it's something you need!

peter-doggart avatar Apr 14 '23 12:04 peter-doggart

Someone has updates on this PR? It would be great if it's merged to the main code base..

gabinguo avatar May 06 '23 22:05 gabinguo