gnosis/lighthouse
1
0
Fork 0
mirror of https://github.com/sigp/lighthouse.git synced 2026-04-19 13:58:28 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add requires --http when using vc subcommands --http-port (#7405)

Browse Source 
Prevent running  `lighthouse vc --http-port <PORT>` without `--http`.
Issue: https://github.com/sigp/lighthouse/issues/7402


  Added requires `--http` when using `lighthouse vc --http-port <PORT>`.
Implemented a test code for this issue.
This commit is contained in:
Yeongjong Pyo
2025-05-07 14:00:56 +09:00
committed by GitHub
parent 3d92e3663b
commit 058dae0641
2 changed files with 9 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
lighthouse/tests/validator_client.rs
View File

@@ -317,6 +317,14 @@ fn missing_unencrypted_http_transport_flag() {
.with_config(|config| assert_eq!(config.http_api.listen_addr, addr)); .with_config(|config| assert_eq!(config.http_api.listen_addr, addr));
} }
#[test] #[test]
#[should_panic]
fn missing_http_http_port_flag() {
CommandLineTest::new()
.flag("http-port", Some("9090"))
.run()
.with_config(|config| assert_eq!(config.http_api.listen_port, 9090));
}
#[test]
fn http_port_flag() { fn http_port_flag() {
CommandLineTest::new() CommandLineTest::new()
.flag("http", None) .flag("http", None)

1
validator_client/src/cli.rs
View File

@@ -219,6 +219,7 @@ pub struct ValidatorClient {
#[clap( #[clap(
long, long,
requires = "http",
value_name = "PORT", value_name = "PORT",
default_value_t = 5062, default_value_t = 5062,
help = "Set the listen TCP port for the RESTful HTTP API server.", help = "Set the listen TCP port for the RESTful HTTP API server.",