Ren "websocket" config to "socket", update def ports help.

Author: evoskuil

include/bitcoin/node/channels/channel_websocket.hpp

@@ -28,6 +28,7 @@ namespace libbitcoin {
 namespace node {
 
 /// Abstract base websocket channel state for the node.
+/// Does not inherit node::channel_http but does inherit network::channel_http.
 class BCN_API channel_websocket
   : public network::channel_websocket,
     public node::channel

include/bitcoin/node/settings.hpp

@@ -136,7 +136,7 @@ class BCN_API settings
     server::settings::html_server explore{ "explore" };
 
     /// native websocket query interface (http/s->tcp/s, json, handshake)
-    network::settings::websocket_server websocket{ "websocket" };
+    network::settings::websocket_server socket{ "socket" };
 
     /// bitcoind compat interface (http/s, stateless json-rpc-v2)
     network::settings::http_server bitcoind{ "bitcoind" };

src/full_node.cpp

@@ -548,7 +548,7 @@ session_explore::ptr full_node::attach_explore_session() NOEXCEPT
 
 session_websocket::ptr full_node::attach_websocket_session() NOEXCEPT
 {
-    return net::attach<session_websocket>(*this, config_.server.websocket);
+    return net::attach<session_websocket>(*this, config_.server.socket);
 }
 
 session_bitcoind::ptr full_node::attach_bitcoind_session() NOEXCEPT

src/parser.cpp

@@ -94,7 +94,7 @@ parser::parser(system::chain::selection context) NOEXCEPT
     // admin
     configured.server.web.binds.emplace_back(asio::address{}, 8080_u16);
     configured.server.explore.binds.emplace_back(asio::address{}, 8180_u16);
-    configured.server.websocket.binds.emplace_back(asio::address{}, 8280_u16);
+    configured.server.socket.binds.emplace_back(asio::address{}, 8280_u16);
     configured.server.bitcoind.binds.emplace_back(asio::address{}, 8380_u16);
     configured.server.electrum.binds.emplace_back(asio::address{}, 8480_u16);
     configured.server.stratum_v1.binds.emplace_back(asio::address{}, 8580_u16);
@@ -1149,7 +1149,7 @@ options_metadata parser::load_settings() THROWS
     (
         "explore.bind",
         value<network::config::authorities>(&configured.server.explore.binds),
-        "IP address to bind, multiple allowed, defaults to '0.0.0.0:8080' (all IPv4)."
+        "IP address to bind, multiple allowed, defaults to '0.0.0.0:8180' (all IPv4)."
     )
     (
         "explore.connections",
@@ -1187,35 +1187,35 @@ options_metadata parser::load_settings() THROWS
         "The path of the default source page, defaults to 'index.html'."
     )
 
-    /* [websocket] */
+    /* [socket] */
     (
         "websocket.secure",
-        value<bool>(&configured.server.websocket.secure),
+        value<bool>(&configured.server.socket.secure),
         "The service requires TLS (not implemented), defaults to 'false'."
     )
     (
-        "websocket.bind",
-        value<network::config::authorities>(&configured.server.websocket.binds),
-        "IP address to bind, multiple allowed, defaults to '0.0.0.0:8080' (all IPv4)."
+        "socket.bind",
+        value<network::config::authorities>(&configured.server.socket.binds),
+        "IP address to bind, multiple allowed, defaults to '0.0.0.0:8280' (all IPv4)."
     )
     (
-        "websocket.connections",
-        value<uint16_t>(&configured.server.websocket.connections),
+        "socket.connections",
+        value<uint16_t>(&configured.server.socket.connections),
         "The required maximum number of connections, defaults to '0'."
     )
     (
-        "websocket.timeout_seconds",
-        value<uint32_t>(&configured.server.websocket.timeout_seconds),
+        "socket.timeout_seconds",
+        value<uint32_t>(&configured.server.socket.timeout_seconds),
         "The idle timeout (http keep-alive), defaults to '60'."
     )
     (
-        "websocket.server",
-        value<std::string>(&configured.server.websocket.server),
+        "socket.server",
+        value<std::string>(&configured.server.socket.server),
         "The server name (http header), defaults to '" BC_HTTP_SERVER_NAME "'."
     )
     (
-        "websocket.host",
-        value<network::config::endpoints>(&configured.server.websocket.hosts),
+        "socket.host",
+        value<network::config::endpoints>(&configured.server.socket.hosts),
         "The host name (http verification), multiple allowed, defaults to empty (disabled)."
     )
 
@@ -1228,7 +1228,7 @@ options_metadata parser::load_settings() THROWS
     (
         "bitcoind.bind",
         value<network::config::authorities>(&configured.server.bitcoind.binds),
-        "IP address to bind, multiple allowed, defaults to '0.0.0.0:8080' (all IPv4)."
+        "IP address to bind, multiple allowed, defaults to '0.0.0.0:8380' (all IPv4)."
     )
     (
         "bitcoind.connections",
@@ -1260,7 +1260,7 @@ options_metadata parser::load_settings() THROWS
     (
         "electrum.bind",
         value<network::config::authorities>(&configured.server.electrum.binds),
-        "IP address to bind, multiple allowed, defaults to '0.0.0.0:8080' (all IPv4)."
+        "IP address to bind, multiple allowed, defaults to '0.0.0.0:8480' (all IPv4)."
     )
     (
         "electrum.connections",
@@ -1282,7 +1282,7 @@ options_metadata parser::load_settings() THROWS
     (
         "stratum_v1.bind",
         value<network::config::authorities>(&configured.server.stratum_v1.binds),
-        "IP address to bind, multiple allowed, defaults to '0.0.0.0:8080' (all IPv4)."
+        "IP address to bind, multiple allowed, defaults to '0.0.0.0:8580' (all IPv4)."
     )
     (
         "stratum_v1.connections",
@@ -1304,7 +1304,7 @@ options_metadata parser::load_settings() THROWS
     (
         "stratum_v2.bind",
         value<network::config::authorities>(&configured.server.stratum_v2.binds),
-        "IP address to bind, multiple allowed, defaults to '0.0.0.0:8080' (all IPv4)."
+        "IP address to bind, multiple allowed, defaults to '0.0.0.0:8680' (all IPv4)."
     )
     (
         "stratum_v2.connections",

test/settings.cpp

@@ -159,10 +159,10 @@ BOOST_AUTO_TEST_CASE(server__explore_server__defaults__expected)
 BOOST_AUTO_TEST_CASE(server__websocket_server__defaults__expected)
 {
     const server::settings instance{};
-    const auto server = instance.websocket;
+    const auto server = instance.socket;
 
     // tcp_server
-    BOOST_REQUIRE_EQUAL(server.name, "websocket");
+    BOOST_REQUIRE_EQUAL(server.name, "socket");
     BOOST_REQUIRE(!server.secure);
     BOOST_REQUIRE(server.binds.empty());
     BOOST_REQUIRE_EQUAL(server.connections, 0u);