fix(rest): change rest server result error type to string

waku-org · Apr 19, 2023 · d5ef933 · d5ef933
1 parent e8dceb2
commit d5ef933
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/waku/v2/node/rest/server.nim b/waku/v2/node/rest/server.nim
@@ -11,7 +11,7 @@ import
   presto
 
 
-type RestServerResult*[T] = Result[T, cstring]
+type RestServerResult*[T] = Result[T, string]
 
 
 ### Configuration
@@ -84,10 +84,11 @@ proc init*(T: type RestServerRef,
       maxHeadersSize = maxHeadersSize,
       maxRequestBodySize = maxRequestBodySize
     )
-  except CatchableError as ex:
-    return err(cstring(ex.msg))
+  except CatchableError:
+    return err(getCurrentExceptionMsg())
 
-  res
+  # RestResult error type is cstring, so we need to map it to string
+  res.mapErr(proc(err: cstring): string = $err)
 
 proc newRestHttpServer*(ip: ValidIpAddress, port: Port,
                         allowedOrigin=none(string),