This fixes an issue where an upload request without a Content-Type in
the file's multipart part would prevent the upload and throw
missleading errors. This patch removes the requirement and ignores
all multipart headers once the needed information has been extracted.

I have tested these changes against a 7.2-11 server and both a previously
broken upload method (without the Content-Type) and using the webUI in
Chrome (which includes a Content-Type) correctly uploads the file.

Changes since v1:
* remove `xx` and escaping of spaces from regex

John Hollowell (2):
  fix #4344: http-server: ignore unused multipart headers
  Remove whitespace ignore from regex

 src/PVE/APIServer/AnyEvent.pm | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

--
2.30.2

_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to