Skip to content
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

Response Content-Type header incorrectly contains the quality qualifier. #13

Closed
MarkKharitonov opened this issue Aug 1, 2011 · 3 comments
Labels
Milestone

Comments

@MarkKharitonov
Copy link
Contributor

Hi, here is a response I get from OpenRasta:

HTTP/1.1 200 OK
Content-Length: 189
Content-Type: application/xml; q="0,9"
Server: Microsoft-HTTPAPI/2.0
Date: Mon, 01 Aug 2011 08:43:37 GMT

<?xml version="1.0" encoding="utf-8"?>
<Home xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance">
  <Title>Welcome home.</Title>
</Home>

Notice the Content-Type header. It should only have application/xml, without the quality part, having which there makes it more difficult to match deserializer based on the Content-Type.

The funny thing is that I am sure I have seent someone reporting this issue and Seb answering him that it is a bug and it was fixed. Well, it is not fixed in openrasta-core.

@MarkKharitonov
Copy link
Contributor Author

I think I know what is the problem. I am currently using HttpListener based host and I guess the fix missed it. So, I just patched the OpenRasta.Hosting.HttpListener.HttpListenerResponse.WriteHeaders method and it is working OK now.

@MarkKharitonov
Copy link
Contributor Author

Nope, I was wrong, OpenRasta.Hosting.AspNet.AspNetResponse.WriteHeaders is affected as well.

HenryKeen pushed a commit to HenryKeen/openrasta-core that referenced this issue Oct 7, 2014
Fixed problem nunit console doing shadow copy
@serialseb
Copy link
Member

Fixed in other parts of the code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants