Client: add -protocol to the client command line.

This commit is contained in:
Pietro Abate 2017-11-23 15:57:27 +01:00 committed by Benjamin Canou
parent f2c27c78ab
commit 4c30016af3
2 changed files with 71 additions and 14 deletions

View File

@ -10,19 +10,31 @@
(* Tezos Command line interface - Configuration and Arguments Parsing *) (* Tezos Command line interface - Configuration and Arguments Parsing *)
type error += Invalid_block_argument of string type error += Invalid_block_argument of string
type error += Invalid_protocol_argument of string
type error += Invalid_port_arg of string type error += Invalid_port_arg of string
let () = let () =
register_error_kind register_error_kind
`Branch `Branch
~id: "badBlocksArgument" ~id: "badBlockArgument"
~title: "Bad Blocks Argument" ~title: "Bad Block Argument"
~description: "Blocks argument could not be parsed" ~description: "Block argument could not be parsed"
~pp: ~pp:
(fun ppf s -> (fun ppf s ->
Format.fprintf ppf "Value provided for -block flag (%s) could not be parsed" s) Format.fprintf ppf "Value provided for -block flag (%s) could not be parsed" s)
Data_encoding.(obj1 (req "value" string)) Data_encoding.(obj1 (req "value" string))
(function Invalid_block_argument s -> Some s | _ -> None) (function Invalid_block_argument s -> Some s | _ -> None)
(fun s -> Invalid_block_argument s) ; (fun s -> Invalid_block_argument s) ;
register_error_kind
`Branch
~id: "badProtocolArgument"
~title: "Bad Protocol Argument"
~description: "Protocol argument could not be parsed"
~pp:
(fun ppf s ->
Format.fprintf ppf "Value provided for -protocol flag (%s) does not correspond to any known protocol" s)
Data_encoding.(obj1 (req "value" string))
(function Invalid_protocol_argument s -> Some s | _ -> None)
(fun s -> Invalid_protocol_argument s) ;
register_error_kind register_error_kind
`Branch `Branch
~id: "invalidPortArgument" ~id: "invalidPortArgument"
@ -93,6 +105,7 @@ end
type cli_args = { type cli_args = {
block: Node_rpc_services.Blocks.block ; block: Node_rpc_services.Blocks.block ;
protocol: Protocol_hash.t option ;
print_timings: bool ; print_timings: bool ;
log_requests: bool ; log_requests: bool ;
force: bool ; force: bool ;
@ -100,6 +113,7 @@ type cli_args = {
let default_cli_args = { let default_cli_args = {
block = Client_commands.default_cfg.block ; block = Client_commands.default_cfg.block ;
protocol = None ;
print_timings = false ; print_timings = false ;
log_requests = false ; log_requests = false ;
force = false ; force = false ;
@ -113,10 +127,22 @@ let string_parameter : (string, Client_commands.context) parameter =
let block_parameter = let block_parameter =
parameter parameter
(fun _ block -> match Node_rpc_services.Blocks.parse_block block with (fun _ block -> match Node_rpc_services.Blocks.parse_block block with
| Error _ -> | Error _ -> fail (Invalid_block_argument block)
fail (Invalid_block_argument block)
| Ok block -> return block) | Ok block -> return block)
let protocol_parameter =
parameter
(fun _ arg ->
try
let (hash,_commands) =
List.find (fun (hash,_commands) ->
(Protocol_hash.to_short_b58check hash) = arg
) (Client_commands.get_versions ())
in
return (Some hash)
with Not_found -> fail (Invalid_protocol_argument arg)
)
(* Command-line only args (not in config file) *) (* Command-line only args (not in config file) *)
let base_dir_arg = let base_dir_arg =
default_arg default_arg
@ -143,6 +169,11 @@ let block_arg =
~doc:"The block on which to apply contextual commands." ~doc:"The block on which to apply contextual commands."
~default:(Node_rpc_services.Blocks.to_string default_cli_args.block) ~default:(Node_rpc_services.Blocks.to_string default_cli_args.block)
block_parameter block_parameter
let protocol_arg =
arg
~parameter:"-protocol"
~doc:"Use contextual commands of a specific protocol."
protocol_parameter
let log_requests_switch = let log_requests_switch =
switch switch
~parameter:"-log-requests" ~parameter:"-log-requests"
@ -171,11 +202,12 @@ let tls_switch =
~doc:"Use TLS to connect to node." ~doc:"Use TLS to connect to node."
let global_options = let global_options =
args9 base_dir_arg args10 base_dir_arg
config_file_arg config_file_arg
force_switch force_switch
timings_switch timings_switch
block_arg block_arg
protocol_arg
log_requests_switch log_requests_switch
addr_arg addr_arg
port_arg port_arg
@ -191,6 +223,7 @@ let parse_config_args (ctx : Client_commands.context) argv =
force, force,
timings, timings,
block, block,
protocol,
log_requests, log_requests,
node_addr, node_addr,
node_port, node_port,
@ -200,6 +233,11 @@ let parse_config_args (ctx : Client_commands.context) argv =
| None -> base_dir // "config" | None -> base_dir // "config"
| Some config_file -> config_file in | Some config_file -> config_file in
let config_dir = Filename.dirname config_file in let config_dir = Filename.dirname config_file in
let protocol =
match protocol with
| None -> None
| Some p -> p
in
let cfg = let cfg =
if not (Sys.file_exists config_file) then if not (Sys.file_exists config_file) then
{ Cfg_file.default with base_dir = base_dir } { Cfg_file.default with base_dir = base_dir }
@ -241,4 +279,4 @@ let parse_config_args (ctx : Client_commands.context) argv =
end ; end ;
IO.mkdir config_dir ; IO.mkdir config_dir ;
if not (Sys.file_exists config_file) then Cfg_file.write config_file cfg ; if not (Sys.file_exists config_file) then Cfg_file.write config_file cfg ;
(cfg, { block ; print_timings = timings ; log_requests ; force }, remaining) (cfg, { block ; print_timings = timings ; log_requests ; force ; protocol }, remaining)

View File

@ -53,13 +53,32 @@ let main () =
} in } in
begin begin
Client_node_rpcs.Blocks.protocol rpc_config parsed_args.block >>= function Client_node_rpcs.Blocks.protocol rpc_config parsed_args.block >>= function
| Ok version -> | Ok version -> begin
match parsed_args.protocol with
| None ->
return (Some version, Client_commands.commands_for_version version) return (Some version, Client_commands.commands_for_version version)
| Error errs -> | Some given_version -> begin
if not (Hash.Protocol_hash.equal version given_version) then
Format.eprintf
"@[<v 2>Warning:@,\
The protocol provided via `-protocol` (%a)@,\
is not the one retrieved from the node (%a).@."
Hash.Protocol_hash.pp_short given_version
Hash.Protocol_hash.pp_short version ;
return (Some version, Client_commands.commands_for_version given_version)
end
end
| Error errs -> begin
match parsed_args.protocol with
| None -> begin
Format.eprintf Format.eprintf
"@[<v 2>Ignored error:@,Failed to acquire the protocol version from the node@,%a@." "@[<v 2>Ignored error:@,Failed to acquire the protocol version from the node@,%a@."
(Format.pp_print_list pp) errs ; (Format.pp_print_list pp) errs ;
return (None, []) return (None, [])
end
| Some version ->
return (Some version, Client_commands.commands_for_version version)
end
end >>=? fun (_version, commands_for_version) -> end >>=? fun (_version, commands_for_version) ->
let commands = let commands =
Client_generic_rpcs.commands @ Client_generic_rpcs.commands @