-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Revert "If HTTP charset header is empty, do not append content encoding ... #1338
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
Conversation
…ng to Content-Type header" This reverts commit 3cc0e3f. This commit break globalization responseEncoding setting see bug 23810 in xamarin bugzilla Signed-off-by: Etienne CHAMPETIER <[email protected]>
|
Hello! I'm the build bot for the Mono project. I need approval from a Mono team member to build this pull request. A team member should reply with "approve" to approve a build of this pull request, "whitelist" to whitelist this and all future pull requests from this contributor, or "build" to explicitly request a build, even if one has already been done. Contributors can ignore this message. |
|
@echampet I'm sorry, but this commit DOES require a unit test in order to be accepted. If there's none we risk a regression |
|
@grendello, i know, but still don't have time |
|
Sorry about that, but until someone comes up with a test, this commit cannot be accepted... |
|
I don't think reverting this is the right way. The original PR #866 included some unit tests that were fixed by the commit so reverting it will make them fail. We should instead see under what conditions we should fallback to |
|
Please notify mono-deve-list and discuss this PR there. |
|
What's the status of this? It's hard to justify accepting this without a test case. |
|
hi @alexrp, reverting is not the way to go but there is still a regression with 3cc0e3f haven't looked for a while if it was fixed by someone else |
|
@echampet I'm closing this, if you have time to verify that the regression is still there, it would be great if you can document that on bugzilla and ideally submit another PR with the proper fix+test :) |
|
hi @lewurm |
It should be marked as NEEDINFO because you didn't provide a testcase or detailed steps to reproduce. |
|
@knocte my ticket isn't crystal clear but the problem is dead simple |
...to Content-Type header"
This reverts commit 3cc0e3f.
This commit break globalization responseEncoding setting
see bug 23810 in xamarin bugzilla
Signed-off-by: Etienne CHAMPETIER [email protected]