diff options
| author | moneromooo-monero <moneromooo-monero@users.noreply.github.com> | 2023-02-07 12:12:02 +0000 |
|---|---|---|
| committer | moneromooo-monero <moneromooo-monero@users.noreply.github.com> | 2023-02-07 12:12:02 +0000 |
| commit | ac9a8978cab3fe89d0a60b72f53d46ea5b9f751f (patch) | |
| tree | 5d546d0b634828a42697ec18af14ec23eb1cacf9 | |
| parent | Merge pull request #8713 (diff) | |
| download | monero-ac9a8978cab3fe89d0a60b72f53d46ea5b9f751f.tar.xz | |
Revert "epee: when loading a JSON RPC payload, ensure params is an object"
This reverts commit b67e931843cd0c8191ec49f80448c010cba39901.
This turns out to be a bad idea, because we do have weird RPC which
do actually go counter to this
| -rw-r--r-- | contrib/epee/include/net/http_server_handlers_map2.h | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/contrib/epee/include/net/http_server_handlers_map2.h b/contrib/epee/include/net/http_server_handlers_map2.h index 00967b2da..848b8ffc4 100644 --- a/contrib/epee/include/net/http_server_handlers_map2.h +++ b/contrib/epee/include/net/http_server_handlers_map2.h @@ -152,16 +152,6 @@ epee::serialization::store_t_to_json(static_cast<epee::json_rpc::error_response&>(rsp), response_info.m_body); \ return true; \ } \ - epee::serialization::storage_entry se; \ - if(ps.get_value("params", se, nullptr) && se.type() != typeid(epee::serialization::section)) \ - { \ - boost::value_initialized<epee::json_rpc::error_response> rsp; \ - static_cast<epee::json_rpc::error_response&>(rsp).jsonrpc = "2.0"; \ - static_cast<epee::json_rpc::error_response&>(rsp).error.code = -32701; \ - static_cast<epee::json_rpc::error_response&>(rsp).error.message = "Params is not an object"; \ - epee::serialization::store_t_to_json(static_cast<epee::json_rpc::error_response&>(rsp), response_info.m_body); \ - return true; \ - } \ epee::serialization::storage_entry id_; \ id_ = epee::serialization::storage_entry(std::string()); \ ps.get_value("id", id_, nullptr); \ |