doc: Drop (default: none) from -i2psam description
What the developer wrote
Authored by Ryan Ofsky
doc: Drop (default: none) from -i2psam description
Suggested by Vojtěch Strnad (vostrnad)
https://github.com/bitcoin/bitcoin/pull/33770#discussion_r2529117675
What changed, and why it matters
This commit is a one-word documentation cleanup. It removes the phrase '(default: none)' from the help text describing the -i2psam command-line option in Bitcoin Core. The actual behavior of the software does not change; only the user-visible description is slightly simplified for consistency with other options.
No security action needed. This is a benign documentation-only change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
In src/init.cpp, the ArgsManager::AddArg call for ‘-i2psam=
Changed components
src/init.cpp-i2psam help textInspect captured patch +1 / −1
diff --git a/src/init.cpp b/src/init.cpp
index d3139f02..7e014eb5 100644
--- a/src/init.cpp
+++ b/src/init.cpp
@@ -553,7 +553,7 @@ void SetupServerArgs(ArgsManager& argsman, bool can_listen_ipc)
#else
argsman.AddArg("-onion=<ip:port>", "Use separate SOCKS5 proxy to reach peers via Tor onion services, set -noonion to disable (default: -proxy)", ArgsManager::ALLOW_ANY, OptionsCategory::CONNECTION);
#endif
- argsman.AddArg("-i2psam=<ip:port>", "I2P SAM proxy to reach I2P peers and accept I2P connections (default: none)", ArgsManager::ALLOW_ANY, OptionsCategory::CONNECTION);
+ argsman.AddArg("-i2psam=<ip:port>", "I2P SAM proxy to reach I2P peers and accept I2P connections", ArgsManager::ALLOW_ANY, OptionsCategory::CONNECTION);
argsman.AddArg("-i2pacceptincoming", strprintf("Whether to accept inbound I2P connections (default: %i). Ignored if -i2psam is not set. Listening for inbound I2P connections is done through the SAM proxy, not by binding to a local address and port.", DEFAULT_I2P_ACCEPT_INCOMING), ArgsManager::ALLOW_ANY, OptionsCategory::CONNECTION);
argsman.AddArg("-onlynet=<net>", "Make automatic outbound connections only to network <net> (" + Join(GetNetworkNames(), ", ") + "). Inbound and manual connections are not affected by this option. It can be specified multiple times to allow multiple networks.", ArgsManager::ALLOW_ANY, OptionsCategory::CONNECTION);
argsman.AddArg("-v2transport", strprintf("Support v2 transport (default: %u)", DEFAULT_V2_TRANSPORT), ArgsManager::ALLOW_ANY, OptionsCategory::CONNECTION);
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.