Let Werkzeug create a proper response string for CORS requests #359
Conversation
|
+1 for the pull request. Works fine under python 2.7 with isso, gevent and nginx. |
|
The edits also works under python 3.5.3 with isso, gevent and nginx. |
| add_cors_headers("200 Ok", [("Content-Type", "text/plain")]) | ||
| return [] | ||
| response = Response() | ||
| return response(environ, add_cors_headers) |
There was a problem hiding this comment.
I don't see why this needs to be changed - the issue is the first argument to werkzeug.datastructures.Headers.
| add_cors_headers("200 Ok", [("Content-Type", "text/plain")]) | ||
| return [] | ||
| response = Response() | ||
| return response(environ, add_cors_headers) |
There was a problem hiding this comment.
I don't see why this needs to be changed - the issue is the first argument to werkzeug.datastructures.Headers.
|
Okay, just fixed it: #555 . |
ix5
added
needs-decision
|
Is anyone still using Closing soon unless someone can provide more information. |
|
I forgot that I had already looked into this in #297 (comment) Closing. |
ix5
removed
the
needs-decision
This fixes a Unicode issue in
wsgi.pyand resolves #297. It should also resolve #347.