mirrors/core-lightning
1
0
Fork 0
mirror of https://github.com/ElementsProject/lightning.git synced 2025-03-04 03:03:51 +01:00
Code Issues Projects Releases Wiki Activity

lightningd: require "jsonrpc": "2.0" as per JSONRPC spec.

Browse source 
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Changelog-Removed: JSON-RPC: require the `"jsonrpc": "2.0"` property (requests without this deprecated in v0.10.2).
This commit is contained in:
Rusty Russell 2023-03-14 15:52:50 +10:30
parent 9366e6b39f
commit 658bae30d5
1 changed files with 10 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
lightningd/jsonrpc.c
View file

@ -707,7 +707,7 @@ static void replace_command(struct rpc_command_hook_payload *p,
const char *buffer, const char *buffer,
const jsmntok_t *replacetok) const jsmntok_t *replacetok)
{ {
const jsmntok_t *method = NULL, *params = NULL; const jsmntok_t *method = NULL, *params = NULL, *jsonrpc;
const char *bad; const char *bad;
/* Must contain "method", "params" and "id" */ /* Must contain "method", "params" and "id" */
@ -739,15 +739,11 @@ static void replace_command(struct rpc_command_hook_payload *p,
goto fail; goto fail;
} }
// deprecated phase to give the possibility to all to migrate and stay safe jsonrpc = json_get_member(buffer, replacetok, "jsonrpc");
// from this more restrictive change.
if (!deprecated_apis) {
const jsmntok_t *jsonrpc = json_get_member(buffer, replacetok, "jsonrpc");
if (!jsonrpc || jsonrpc->type != JSMN_STRING || !json_tok_streq(buffer, jsonrpc, "2.0")) { if (!jsonrpc || jsonrpc->type != JSMN_STRING || !json_tok_streq(buffer, jsonrpc, "2.0")) {
bad = "jsonrpc: \"2.0\" must be specified in the request"; bad = "jsonrpc: \"2.0\" must be specified in the request";
goto fail; goto fail;
} }
}
was_pending(command_exec(p->cmd->jcon, p->cmd, buffer, replacetok, was_pending(command_exec(p->cmd->jcon, p->cmd, buffer, replacetok,
params)); params));
@ -883,7 +879,7 @@ REGISTER_PLUGIN_HOOK(rpc_command,
static struct command_result * static struct command_result *
parse_request(struct json_connection *jcon, const jsmntok_t tok[]) parse_request(struct json_connection *jcon, const jsmntok_t tok[])
{ {
const jsmntok_t *method, *id, *params, *filter; const jsmntok_t *method, *id, *params, *filter, *jsonrpc;
struct command *c; struct command *c;
struct rpc_command_hook_payload *rpc_hook; struct rpc_command_hook_payload *rpc_hook;
bool completed; bool completed;
@ -910,16 +906,11 @@ parse_request(struct json_connection *jcon, const jsmntok_t tok[])
return NULL; return NULL;
} }
// Adding a deprecated phase to make sure that all the Core Lightning wrapper jsonrpc = json_get_member(jcon->buffer, tok, "jsonrpc");
// can migrate all the frameworks
if (!deprecated_apis) {
const jsmntok_t *jsonrpc = json_get_member(jcon->buffer, tok, "jsonrpc");
if (!jsonrpc || jsonrpc->type != JSMN_STRING || !json_tok_streq(jcon->buffer, jsonrpc, "2.0")) { if (!jsonrpc || jsonrpc->type != JSMN_STRING || !json_tok_streq(jcon->buffer, jsonrpc, "2.0")) {
json_command_malformed(jcon, "null", "jsonrpc: \"2.0\" must be specified in the request"); json_command_malformed(jcon, "null", "jsonrpc: \"2.0\" must be specified in the request");
return NULL; return NULL;
} }
}
/* Allocate the command off of the `jsonrpc` object and not /* Allocate the command off of the `jsonrpc` object and not
* the connection since the command may outlive `conn`. */ * the connection since the command may outlive `conn`. */