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.

Sign up for GitHub

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

Jump to bottom

test_multipart_files_content & test_multipart_data_and_files_content fail with aiohttp 3.9.4 #35220

Closed

xiangyan99 opened this issue Apr 15, 2024 · 1 comment · Fixed by #35223

Labels

Azure.Core Client

Member

xiangyan99 commented Apr 15, 2024 •

edited

Loading

xiangyan99 added Client Azure.Core labels

Member

pvaneck commented Apr 15, 2024

Seems like this change: aio-libs/aiohttp@cebe526#diff-d582bf292efb8e19696d88c895b99e0937687cb909d9d00b5c2f1d948a5cbae5R873 where they no longer add the Content-Length header in payloads for multipart/form-data.

The motivation was to conform to this RFC: https://datatracker.ietf.org/doc/html/rfc7578#section-4.8 where the Content-Length header is not allowed.

pvaneck mentioned this issue

[Core][Corehttp] Update multipart test-server code #35223

Merged

xiangyan99 closed this as completed in #35223

github-actions bot locked and limited conversation to collaborators

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.