HttpAuthHook use async raises

status-im · Jan 27, 2024 · c5c40af · c5c40af
1 parent addfc32
commit c5c40af
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/json_rpc/servers/httpserver.nim b/json_rpc/servers/httpserver.nim
@@ -32,7 +32,7 @@ type
   # - HttpResponse: could not authenticate, stop execution
   #   and return the response
   HttpAuthHook* = proc(request: HttpRequestRef): Future[HttpResponseRef]
-                  {.gcsafe, raises: [Defect, CatchableError].}
+                  {.gcsafe, async: (raises: [CatchableError]).}
 
   # This inheritance arrangement is useful for
   # e.g. combo HTTP server

diff --git a/tests/testhook.nim b/tests/testhook.nim
@@ -25,7 +25,8 @@ proc authHeaders(): seq[(string, string)] =
   @[("Auth-Token", "Good Token")]
 
 suite "HTTP server hook test":
-  proc mockAuth(req: HttpRequestRef): Future[HttpResponseRef] {.async.} =
+  proc mockAuth(req: HttpRequestRef): Future[HttpResponseRef] {.
+        gcsafe, async: (raises: [CatchableError]).} =
     if req.headers.getString("Auth-Token") == "Good Token":
       return HttpResponseRef(nil)