Merge pull request #11 from slamdata/onerror-handler

jdegoes · jdegoes · commit 8fbf208b0f5c · 2015-04-13T09:42:52.000-06:00
Better handler for XHR onerror
diff --git a/src/Network/HTTP/Affjax.purs b/src/Network/HTTP/Affjax.purs
@@ -161,7 +161,7 @@ foreign import unsafeAjax
         }
       }
       xhr.onerror = function (err) {
-        errback(err)();
+        errback(new Error("AJAX request failed: " + options.method + " " + options.url))();
       };
       xhr.onload = function () {
         callback({
diff --git a/test/Main.purs b/test/Main.purs
@@ -34,3 +34,6 @@ main = launchAff $ do
 
   res <- attempt $ get "/arrayview"
   liftEff $ either traceAny (traceAny :: AffjaxResponse Foreign -> _) res
+
+  res <- attempt $ get "ttp://www.google.com"
+  liftEff $ either traceAny (traceAny :: AffjaxResponse Foreign -> _) res

Original file line number	Diff line number	Diff line change
`@@ -161,7 +161,7 @@ foreign import unsafeAjax`
`161`	`161`	`}`
`162`	`162`	`}`
`163`	`163`	`xhr.onerror = function (err) {`
`164`		`- errback(err)();`
	`164`	`+ errback(new Error("AJAX request failed: " + options.method + " " + options.url))();`
`165`	`165`	`};`
`166`	`166`	`xhr.onload = function () {`
`167`	`167`	`callback({`