-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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.
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
Fix post MultiPart #47006
Fix post MultiPart #47006
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I also confirm removing the block fixed things.
Hurrah for the additional test.
This reverts commit c62a255. Fixes qgis#46989
Can you please wait to merge? You have disabled a test about another problem that this code was fixing. |
Happy to wait for you @elpaso |
@elpaso can you be more precise about the problem?
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, I checked the action manually and it works as expected, I added that piece of code to be able to test the action but obviously it had unwanted side effects.
aea92d3
to
b7157db
Compare
This reverts commit c62a255.
Calling
readAll()
onoutgoingData
breaks multipart post requests.Fixes #46989