[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[tor-commits] [Git][tpo/applications/tor-browser][tor-browser-102.12.0esr-12.5-1] fixup! Bug 40597: Implement TorSettings module

Title: GitLab

Pier Angelo Vendrame pushed to branch tor-browser-102.12.0esr-12.5-1 at The Tor Project / Applications / Tor Browser

Commits:

  • 014f6414
    by Pier Angelo Vendrame at 2023-06-01T22:03:04+02:00 
    fixup! Bug 40597: Implement TorSettings module

Bug 41801: Fix handleProcessReady in TorSettings.init

1 changed file:

Changes:

  • browser/modules/TorSettings.jsm
    ... ... @@ -299,7 +299,7 @@ const TorSettings = (() => {
    299 299 
             Services.obs.addObserver(this, TorTopics.ProcessIsReady);
    300 300
    301 301 
             if (TorMonitorService.isRunning) {
    302 
    -          handleProcessReady();
    302 
    +          this.handleProcessReady();
    303 303 
             }
    304 304 
           }
    305 305 
         },
    ... ... @@ -308,22 +308,22 @@ const TorSettings = (() => {
    308 308 
         async observe(subject, topic, data) {
    309 309 
           console.log(`TorSettings: Observed ${topic}`);
    310 310
    311 
    -      // once the tor daemon is ready, we need to apply our settings
    312 
    -      const handleProcessReady = async () => {
    313 
    -        // push down settings to tor
    314 
    -        await this.applySettings();
    315 
    -        console.log("TorSettings: Ready");
    316 
    -        Services.obs.notifyObservers(null, TorSettingsTopics.Ready);
    317 
    -      };
    318 
    -
    319 311 
           switch (topic) {
    320 312 
             case TorTopics.ProcessIsReady:
    321 313 
               Services.obs.removeObserver(this, TorTopics.ProcessIsReady);
    322 
    -          await handleProcessReady();
    314 
    +          await this.handleProcessReady();
    323 315 
               break;
    324 316 
           }
    325 317 
         },
    326 318
    319 
    +    // once the tor daemon is ready, we need to apply our settings
    320 
    +    async handleProcessReady() {
    321 
    +      // push down settings to tor
    322 
    +      await this.applySettings();
    323 
    +      console.log("TorSettings: Ready");
    324 
    +      Services.obs.notifyObservers(null, TorSettingsTopics.Ready);
    325 
    +    },
    326 
    +
    327 327 
         // load our settings from prefs
    328 328 
         loadFromPrefs() {
    329 329 
           console.log("TorSettings: loadFromPrefs()");

    • —
    View it on GitLab.
    You're receiving this email because of your account on gitlab.torproject.org. Manage all notifications · Help 

    _______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits