Change tor config before is launched

to don't get error about options that can't be changed once it's launched.

Author: juga0

bwscanner/attacher.py

@@ -158,30 +158,33 @@ def connect_to_tor(launch_tor, circuit_build_timeout, control_port=None,
     if tor_overrides:
         tor_options.update(tor_overrides)
 
+    tor_config = txtorcon.TorConfig()
+
+    # Update Tor config options from dictionary
+    for key, value in tor_options.items():
+        setattr(tor_config, key, value)
+
     if launch_tor:
         log.info("Spawning a new Tor instance.")
         # TODO: Pass in data_dir directory so consensus can be cached
-        tor = yield txtorcon.launch(reactor)
+        # Launch tor with config, in order to don't get CONF_CHANGED when
+        # updating options that can't be changed while tor is running.
+        tor = yield txtorcon.launch(reactor, _tor_config=tor_config)
     else:
         log.info("Trying to connect to a running Tor instance.")
         if control_port:
             endpoint = endpoints.TCP4ClientEndpoint(reactor, "localhost", control_port)
         else:
             endpoint = None
         tor = yield txtorcon.connect(reactor, endpoint)
+        # TODO: check whether CONF_CHANGED will happen here or not because
+        # we get the state later
+        tor.config = tor_config
+        tor.config.save()
 
-    # Get Tor state first to avoid a race conditions where CONF_CHANGED
-    # messages are received while Txtorcon is reading the consensus.
-    tor_state = yield tor.create_state()
-
-    # Get current TorConfig object
-    tor_config = yield tor.get_config()
     wait_for_consensus = options_need_new_consensus(tor_config, tor_options)
 
-    # Update Tor config options from dictionary
-    for key, value in tor_options.items():
-        setattr(tor_config, key, value)
-    yield tor_config.save()  # Send updated options to Tor
+    tor_state = yield tor.create_state()
 
     if wait_for_consensus:
         yield wait_for_newconsensus(tor_state)