[tbb-commits] [Git][tpo/applications/mullvad-browser][mullvad-browser-115.15.0esr-13.5-1] Bug 1909099 - Always clean up old session restore and sync log files....

[ma1 (@ma1)]

ma1 pushed to branch mullvad-browser-115.15.0esr-13.5-1 at The Tor Project / Applications / Mullvad Browser


Commits:
0dab02ba by Sam Foster at 2024-08-31T13:05:25+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:

- services/common/logmanager.sys.mjs


Changes:

=====================================
services/common/logmanager.sys.mjs
=====================================
@@ -363,12 +363,7 @@ LogManager.prototype = {
         filename,
         this._log
       );
-      // It's not completely clear to markh why we only do log cleanups
-      // for errors, but for now the Sync semantics have been copied...
-      // (one theory is that only cleaning up on error makes it less
-      // likely old error logs would be removed, but that's not true if
-      // there are occasional errors - let's address this later!)
-      if (reason == this.ERROR_LOG_WRITTEN && !this._cleaningUpFileLogs) {
+      if (!this._cleaningUpFileLogs) {
         this._log.trace("Running cleanup.");
         try {
           await this.cleanupLogs();



View it on GitLab: https://gitlab.torproject.org/tpo/applications/mullvad-browser/-/commit/0dab02ba16493fca62d40d2bcfb321583e569d21

-- 
View it on GitLab: https://gitlab.torproject.org/tpo/applications/mullvad-browser/-/commit/0dab02ba16493fca62d40d2bcfb321583e569d21
You're receiving this email because of your account on gitlab.torproject.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.torproject.org/pipermail/tbb-commits/attachments/20240831/5de28e88/attachment-0001.htm>