logos-storage/nim-json-rpc
1
0
Fork 0
mirror of https://github.com/logos-storage/nim-json-rpc.git synced 2026-02-21 06:03:13 +00:00
Code Issues Packages Projects Releases Wiki Activity

Added the ability to handle omitted params fields in incoming requests, and a unit test for it (#94)

Browse Source
This commit is contained in:
Joe Clapis 2021-02-07 14:27:34 -05:00 committed by GitHub
parent b0e82cb70a
commit 831471f6d4
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 14 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
json_rpc/router.nim
View File

@ -71,6 +71,9 @@ template jsonValid*(jsonString: string, node: var JsonNode): (bool, string) =
msg = ""
try:
node = parseJson(line)
# Handle cases where params is omitted
if not node.hasKey(paramsField):
node.add(paramsField, newJArray())
except CatchableError as exc:
valid = false
msg = exc.msg

11
tests/testhttp.nim
View File

@ -56,6 +56,13 @@ const
"Connection: close\r\n" &
"\r\n" &
"{\"jsonrpc\":\"2.0\",\"method\":\"myProc\",\"params\":[\"abc\", [1, 2, 3]],\"id\":67}",
"GET / HTTP/1.1\r\n" &
"Host: www.google.com\r\n" &
"Content-Length: 49\r\n" &
"Content-Type: application/json\r\n" &
"Connection: close\r\n" &
"\r\n" &
"{\"jsonrpc\":\"2.0\",\"method\":\"noParamsProc\",\"id\":67}",
]
proc continuousTest(address: string, port: Port): Future[int] {.async.} =
@ -150,6 +157,8 @@ var httpsrv = newRpcHttpServer(["localhost:8545"])
# Create RPC on server
httpsrv.rpc("myProc") do(input: string, data: array[0..3, int]):
result = %("Hello " & input & " data: " & $data)
httpsrv.rpc("noParamsProc") do():
result = %("Hello world")
httpsrv.start()
@ -176,6 +185,8 @@ suite "HTTP Server/HTTP Client RPC test suite":
waitFor(disconTest("localhost", Port(8545), 6, 200)) == true
test "[Connection]: close test":
check waitFor(disconTest("localhost", Port(8545), 7, 200)) == true
test "Omitted params test":
check waitFor(simpleTest("localhost", Port(8545), 8, 200)) == true
httpsrv.stop()
waitFor httpsrv.closeWait()