Commit 11155c42 authored by Ray Schamp's avatar Ray Schamp Committed by GitHub

Merge pull request #661 from rschamp/bugfix/654

Fix JSON response parsing in IE
parents bfd8bcd6 b89d6445
...@@ -54,6 +54,16 @@ module.exports = function (opts, callback) { ...@@ -54,6 +54,16 @@ module.exports = function (opts, callback) {
} }
xhr(opts, function (err, res, body) { xhr(opts, function (err, res, body) {
if (err) log.error(err); if (err) log.error(err);
if (opts.responseType === 'json' && typeof body === 'string') {
// IE doesn't parse responses as JSON without the json attribute,
// even with responseType: 'json'.
// See https://github.com/Raynos/xhr/issues/123
try {
body = JSON.parse(body);
} catch (e) {
// Not parseable anyway, don't worry about it
}
}
// Legacy API responses come as lists, and indicate to redirect the client like // Legacy API responses come as lists, and indicate to redirect the client like
// [{success: true, redirect: "/location/to/redirect"}] // [{success: true, redirect: "/location/to/redirect"}]
try { try {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment