1.3.0 - Make simple fetch throw errors on non 200 responses.

puppy.nimble

@@ -1,4 +1,4 @@
-version     = "1.2.1"
+version     = "1.3.0"
 author      = "Andre von Houck"
 description = "Puppy fetches HTML pages for Nim."
 license     = "MIT"

src/puppy.nim

@@ -1,4 +1,4 @@
-import puppy/common, net, strutils, urlly, zippy
+import net, puppy/common, strutils, urlly, zippy
 
 export common, urlly
 
@@ -270,9 +270,12 @@ proc newRequest*(
   result.headers.merge(headers)
   result.timeout = timeout
 
-proc fetch*(url: string, verb = "get", headers = newSeq[Header]()): string =
+proc fetch*(url: string, headers = newSeq[Header]()): string =
   let
-    req = newRequest(url, verb, headers)
+    req = newRequest(url, "get", headers)
     res = req.fetch()
   if res.code == 200:
     return res.body
+  raise newException(PuppyError,
+    "Non 200 response code: " & $res.code & "\n" & res.body
+  )

tests/debug_client.nim

@@ -1,11 +1,11 @@
 import puppy, zippy
 
-
 for i in 0 ..< 100:
   block:
     # test basic
     doAssert fetch("http://localhost:8080/ok") == "ok"
-    doAssert fetch("http://localhost:8080/401") == ""
+    doAssertRaises(PuppyError):
+      discard fetch("http://localhost:8080/401")
 
   block:
     # test 404

tests/debug_server.nim

@@ -1,4 +1,4 @@
-import asyncdispatch, asynchttpserver, zippy, uri
+import asyncdispatch, asynchttpserver, uri, zippy
 
 let server = newAsyncHttpServer()