ARCHIVE/clio
1
0
Fork 0
mirror of https://github.com/XRPLF/clio.git synced 2025-11-26 06:35:53 +00:00
Code Issues Packages Projects Releases Wiki Activity

error when marker does not exist (#167)

Browse Source
This commit is contained in:
Nathan Nichols
2022-06-15 16:18:45 -05:00
committed by GitHub
parent 166ff63dbc
commit 14a972c8e2
2 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
src/rpc/RPC.cpp
View File

@@ -145,7 +145,6 @@ static std::unordered_map<std::string, std::function<Result(Context const&)>>
{"ledger_data", &doLedgerData}, {"ledger_data", &doLedgerData},
{"ledger_entry", &doLedgerEntry}, {"ledger_entry", &doLedgerEntry},
{"ledger_range", &doLedgerRange}, {"ledger_range", &doLedgerRange},
{"ledger_data", &doLedgerData},
{"nft_buy_offers", &doNFTBuyOffers}, {"nft_buy_offers", &doNFTBuyOffers},
{"nft_sell_offers", &doNFTSellOffers}, {"nft_sell_offers", &doNFTSellOffers},
{"subscribe", &doSubscribe}, {"subscribe", &doSubscribe},

8
src/rpc/handlers/LedgerData.cpp
View File

@@ -73,6 +73,7 @@ doLedgerData(Context const& context)
return *status; return *status;
auto lgrInfo = std::get<ripple::LedgerInfo>(v); auto lgrInfo = std::get<ripple::LedgerInfo>(v);
boost::json::object header; boost::json::object header;
// no marker means this is the first call, so we return header info // no marker means this is the first call, so we return header info
if (!marker) if (!marker)
@@ -106,6 +107,13 @@ doLedgerData(Context const& context)
response[JS(ledger)] = header; response[JS(ledger)] = header;
} }
} }
else
{
if (!outOfOrder &&
!context.backend->fetchLedgerObject(
*marker, lgrInfo.seq, context.yield))
return Status{Error::rpcINVALID_PARAMS, "markerDoesNotExist"};
}
response[JS(ledger_hash)] = ripple::strHex(lgrInfo.hash); response[JS(ledger_hash)] = ripple::strHex(lgrInfo.hash);
response[JS(ledger_index)] = lgrInfo.seq; response[JS(ledger_index)] = lgrInfo.seq;