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

(MultipartRequest) Content-Transfer-Encoding header is probably unnecessary / wrong #39

Closed
d3v-cl opened this issue Jul 6, 2016 · 2 comments

Comments

@d3v-cl
Copy link

d3v-cl commented Jul 6, 2016

Usually an unnecessary header doesn't hurt. But in my case the browser aborts an xmlhttprequest:

XMLHttpRequest cannot load.... Request header field content-transfer-encoding is not allowed by Access-Control-Allow-Headers in preflight response.

Apparently Content-Transfer-Encoding is not a valid http header anyways:
stackoverflow

Response I got from the server when the Request method was OPTIONS:

HTTP/1.1 204 No Content
Date: Wed, 06 Jul 2016 18:27:29 GMT
Access-Control-Allow-Origin: *
Access-Control-Allow-Methods: GET, POST, DELETE, PUT
Access-Control-Allow-Headers: Content-Type, api_key, Authorization
Allow: OPTIONS,POST
Connection: close
Server: Jetty(9.2.9.v20150224)
@kevmoo
Copy link
Member

kevmoo commented Jul 6, 2016

@nex3 thoughts on this?

@nex3
Copy link
Member

nex3 commented Jul 6, 2016

We definitely shouldn't be passing this as an HTTP header, but it looks like it is necessary as a header for a multipart chunk (see the HTML spec, which refers to RFC 2388).

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

No branches or pull requests

3 participants