diff options
author | stickies-v <stickies-v@protonmail.com> | 2024-04-10 12:16:21 +0200 |
---|---|---|
committer | stickies-v <stickies-v@protonmail.com> | 2024-05-01 14:44:57 +0100 |
commit | 42fb5311b19582361409d65c6fddeadbee14bb97 (patch) | |
tree | 30d0e390d19a61f84583546126273298fdd50fc5 /src/node | |
parent | 842f7fdf786fcbbdf3df40522945813404f8a397 (diff) | |
download | bitcoin-42fb5311b19582361409d65c6fddeadbee14bb97.tar.xz |
rpc: return warnings as an array instead of just a single one
The RPC documentation for `getblockchaininfo`, `getmininginfo` and
`getnetworkinfo` states that "warnings" returns "any network and
blockchain warnings". In practice, only a single warning is returned.
Fix that by returning all warnings as an array.
As a side benefit, cleans up the GetWarnings() logic.
Diffstat (limited to 'src/node')
-rw-r--r-- | src/node/interfaces.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/node/interfaces.cpp b/src/node/interfaces.cpp index 4d2d83812e..bb13e53738 100644 --- a/src/node/interfaces.cpp +++ b/src/node/interfaces.cpp @@ -47,6 +47,7 @@ #include <util/check.h> #include <util/result.h> #include <util/signalinterrupt.h> +#include <util/string.h> #include <util/translation.h> #include <validation.h> #include <validationinterface.h> @@ -91,7 +92,7 @@ public: explicit NodeImpl(NodeContext& context) { setContext(&context); } void initLogging() override { InitLogging(args()); } void initParameterInteraction() override { InitParameterInteraction(args()); } - bilingual_str getWarnings() override { return GetWarnings(true); } + bilingual_str getWarnings() override { return Join(GetWarnings(), Untranslated("<hr />")); } int getExitStatus() override { return Assert(m_context)->exit_status.load(); } uint32_t getLogCategories() override { return LogInstance().GetCategoryMask(); } bool baseInitialize() override |