Verified Commit 7528c21f authored by henry's avatar henry Committed by Pier Angelo Vendrame
Browse files

fixup! TB 40933: Add tor-launcher functionality 

TB 41921: Add a note to TorBootstrapRequest explaining the expectation
that the TorProvider has already read from TorSettings before it
connects.
parent 7350bfda

toolkit/components/tor-launcher/TorBootstrapRequest.sys.mjs

+5 −0
Original line number Diff line number Diff line
@@ -90,6 +90,11 @@ export class TorBootstrapRequest { 
      // Wait for bootstrapping to begin and maybe handle error.

      // Notice that we do not resolve the promise here in case of success, but

      // we do it from the BootstrapStatus observer.

      // NOTE: After TorProviderBuilder.build resolves, TorProvider.init will

      // have completed. In particular, assuming no errors, the TorSettings will

      // have been initialised and passed on to the provider via

      // TorProvider.writeSettings. Therefore we should be safe to immediately

      // call `connect` using the latest user settings.

      lazy.TorProviderBuilder.build()

        .then(provider => provider.connect())

        .catch(err => {