Skip to content

Commit

Permalink
fix(rpc): re-enable default RPC security checks
Browse files Browse the repository at this point in the history
  • Loading branch information
teor2345 committed Feb 21, 2022
1 parent 423f9b0 commit cd6ccdf
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions zebrad/src/components/rpc.rs
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,8 @@ impl RpcServer {
// TODO: use the same tokio executor as the rest of Zebra
//.event_loop_executor(tokio::runtime::Handle::current())
.threads(1)
// TODO: if we enable this security check, does lightwalletd still work?
.allowed_hosts(DomainsValidation::Disabled)
// TODO: disable this security check if we see errors from lightwalletd.
//.allowed_hosts(DomainsValidation::Disabled)
.request_middleware(FixHttpRequestMiddleware)
.start_http(&config.listen_addr)
.expect("Unable to start RPC server");
Expand Down Expand Up @@ -88,8 +88,8 @@ impl RequestMiddleware for FixHttpRequestMiddleware {
});

jsonrpc_http_server::RequestMiddlewareAction::Proceed {
// TODO: if we enable this security check, does lightwalletd still work?
should_continue_on_invalid_cors: true,
// TODO: disable this security check if we see errors from lightwalletd.
should_continue_on_invalid_cors: false,
request,
}
}
Expand Down

0 comments on commit cd6ccdf

Please sign in to comment.