Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@echampet
Copy link

...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]

…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]>
@monoadmin
Copy link

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
Copy link
Author

hi @knocte, sorry, don't have time

@linquize, my PR is reverting one of your commit,
can you take a look to see if you can find a better fix for your problem

@grendello
Copy link
Member

@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

@echampet
Copy link
Author

@grendello, i know, but still don't have time

@grendello
Copy link
Member

Sorry about that, but until someone comes up with a test, this commit cannot be accepted...

@akoeplinger
Copy link
Member

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 ContentEncoding.HeaderName (which reads the globalization responseEncoding setting).

@migueldeicaza
Copy link
Contributor

Please notify mono-deve-list and discuss this PR there.

@echampet
Copy link
Author

@alexrp
Copy link
Contributor

alexrp commented Mar 16, 2015

What's the status of this? It's hard to justify accepting this without a test case.

@echampet
Copy link
Author

hi @alexrp, reverting is not the way to go but there is still a regression with 3cc0e3f
https://bugzilla.xamarin.com/show_bug.cgi?id=23810

haven't looked for a while if it was fixed by someone else

@lewurm
Copy link
Contributor

lewurm commented Mar 28, 2016

@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 :)

@lewurm lewurm closed this Mar 28, 2016
@echampet
Copy link
Author

hi @lewurm
given the commit log (no activity on this file since jan 2015) i'm pretty sure regression is still there
bugzilla bug is still open ...

@knocte
Copy link
Contributor

knocte commented Mar 29, 2016

bugzilla bug is still open

It should be marked as NEEDINFO because you didn't provide a testcase or detailed steps to reproduce.

@echampet
Copy link
Author

@knocte my ticket isn't crystal clear but the problem is dead simple
set <globalization requestEncoding="utf-8" responseEncoding="utf-8"/> in machine.config and look at the http headers of an aspx page, you should see content-type: text/html; charset=utf-8 and not content-type: text/html

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants