fix: fix base_node_service_config not read (#3251)

Description
---
Fixed the base_node_service_config not being initialized with values from the config file.

Motivation and Context
---
See above

How Has This Been Tested?
---
System level testing

applications/tari_base_node/src/bootstrap.rs

@@ -88,7 +88,12 @@ where B: BlockchainBackend + 'static
             pubsub_connector(runtime::Handle::current(), buf_size, config.buffer_rate_limit_base_node);
         let peer_message_subscriptions = Arc::new(peer_message_subscriptions);
 
-        let node_config = BaseNodeServiceConfig::default(); // TODO - make this configurable
+        let node_config = BaseNodeServiceConfig {
+            service_request_timeout: config.service_request_timeout,
+            fetch_blocks_timeout: config.fetch_blocks_timeout,
+            fetch_utxos_timeout: config.fetch_utxos_timeout,
+            ..Default::default()
+        };
         let mempool_config = MempoolServiceConfig::default(); // TODO - make this configurable
 
         let comms_config = self.create_comms_config();

applications/tari_console_wallet/src/init/mod.rs

@@ -325,7 +325,6 @@ pub async fn init_wallet(
         peer_database_name: "peers".to_string(),
         max_concurrent_inbound_tasks: 100,
         outbound_buffer_size: 100,
-        // TODO - make this configurable
         dht: DhtConfig {
             database_url: DbConnectionUrl::File(config.data_dir.join("dht-console-wallet.db")),
             auto_join: true,
@@ -335,7 +334,7 @@ pub async fn init_wallet(
             dedup_cache_capacity: config.dedup_cache_capacity,
             ..Default::default()
         },
-        // TODO: This should be false unless testing locally - make this configurable
+        // This should be false unless testing locally
         allow_test_addresses: config.allow_test_addresses,
         listener_liveness_allowlist_cidrs: Vec::new(),
         listener_liveness_max_sessions: 0,