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

[tor-commits] [Git][tpo/applications/tor-browser][base-browser-128.2.0esr-14.0-1] Bug 1909099 - Always clean up old session restore and sync log files....

Title: GitLab

ma1 pushed to branch base-browser-128.2.0esr-14.0-1 at The Tor Project / Applications / Tor Browser

Commits:

  • dfd03239
    by Sam Foster at 2024-08-31T13:04:35+08:00 
    Bug 1909099 - Always clean up old session restore and sync log files. r=markh,sessionstore-reviewers,dao

Differential Revision: https://phabricator.services.mozilla.com/D217520

1 changed file:

Changes:

  • toolkit/modules/LogManager.sys.mjs
    ... ... @@ -384,12 +384,7 @@ export class LogManager {
    384 384 
             filename,
    385 385 
             this._log
    386 386 
           );
    387 
    -      // It's not completely clear to markh why we only do log cleanups
    388 
    -      // for errors, but for now the Sync semantics have been copied...
    389 
    -      // (one theory is that only cleaning up on error makes it less
    390 
    -      // likely old error logs would be removed, but that's not true if
    391 
    -      // there are occasional errors - let's address this later!)
    392 
    -      if (reason == this.ERROR_LOG_WRITTEN && !this._cleaningUpFileLogs) {
    387 
    +      if (!this._cleaningUpFileLogs) {
    393 388 
             this._log.trace("Running cleanup.");
    394 389 
             try {
    395 390 
               await this.cleanupLogs();

    • —
    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