Dapps and RPC server merge #5365
Conversation
dapps/src/tests/rpc.rs
Outdated
response.assert_status("HTTP/1.1 200 OK"); | ||
assert_eq!(response.body, "31\n{\"jsonrpc\":\"2.0\",\"result\":\"Hello World!\",\"id\":1}\n\n0\n\n".to_owned()); | ||
} | ||
// TODO [Move to RPC tests!] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, just noticed the TODO :)
can't we have the dapps use the "fast minihttp server"? |
@@ -25,7 +25,6 @@ serde_json = "0.9" | |||
app_dirs = "1.1.1" | |||
fdlimit = "0.1" | |||
ws2_32-sys = "0.2" | |||
hyper = { default-features = false, git = "https://github.com/paritytech/hyper" } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉 🎉 🎉
if args.flag_ipc_off { | ||
result.push(Deprecated::ipc_off()); | ||
if args.flag_dapps_pass.is_some() { | ||
result.push(Deprecated::Removed("--dapps-pass")); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How does json-rpc authentication work now? (I was not following your recent prs closely) :p
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nvm, I read pr description
@gavofyork That would require quite some work to re-write current dapps stuff (but might be worth considering in the future). |
Part of #4474