Skip to content

Fix z_listunspent RPC docs, which require different quoting that CLI#1339

Merged
jl777 merged 1 commit intojl777:jl777from
MyHush:z_listunspentdocs
Mar 21, 2019
Merged

Fix z_listunspent RPC docs, which require different quoting that CLI#1339
jl777 merged 1 commit intojl777:jl777from
MyHush:z_listunspentdocs

Conversation

@leto
Copy link
Copy Markdown

@leto leto commented Mar 20, 2019

Fixes #1214

I manually tested this with PIRATE, where I had non-empty z_listunspent output.

Now our RPC docs correctly output the properly-escaped JSON (except for incorrect rpc port for asset chains, which I will report as another issue) for this particular RPC. Others will need to be fixed on a case-by-case basis.

curl --user user --data-binary '{"jsonrpc": "1.0", "id":"curltest", "method": "z_listunspent", "params": [6,9999999,false,["zs14d8tc0hl9q0vg5l28uec5vk6sk34fkj2n8s7jalvw5fxpy6v39yn4s2ga082lymrkjk0x2nqg37","zs14d8tc0hl9q0vg5l28uec5vk6sk34fkj2n8s7jalvw5fxpy6v39yn4s2ga082lymrkjk0x2nqg37"]] }' -H 'content-type: text/plain;' http://127.0.0.1:45453/

@jl777 jl777 merged commit 42200a4 into jl777:jl777 Mar 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants