log backup for errors

Author: Pedro Aim

electron_app/src/database/DBEexporter.js

@@ -596,8 +596,7 @@ const exportLabelTable = async accountId => {
   return formatTableRowsToString(Table.LABEL, labelRows);
 };
 
-const exportEmailTable = async accountId => {
-  const username = myAccount.email;
+const exportEmailTable = async (accountId, email) => {
   let emailRows = [];
   let shouldEnd = false;
   let offset = 0;
@@ -629,14 +628,14 @@ const exportEmailTable = async accountId => {
 
           const body =
             (await getEmailBody({
-              username,
+              username: email,
               metadataKey: newRow.key,
               password: globalManager.databaseKey.get()
             })) ||
             newRow.content ||
             '';
           const headers = await getEmailHeaders({
-            username,
+            username: email,
             metadataKey: newRow.key,
             password: globalManager.databaseKey.get()
           });
@@ -847,6 +846,7 @@ const exportEncryptDatabaseToFile = async ({
   const [recipientId, domain] = accountObj
     ? accountObj.recipientId.split('@')
     : myAccount.recipientId.split('@');
+  const accountEmail = `${recipientId}@${domain || APP_DOMAIN}`;
   const accountId = accountObj ? accountObj.id : myAccount.id;
   const signature =
     accountObj && accountObj.signature !== undefined
@@ -876,7 +876,7 @@ const exportEncryptDatabaseToFile = async ({
   handleProgressCallback(
     exportProgress,
     'saving_account',
-    `${recipientId}@${domain || APP_DOMAIN}`,
+    accountEmail,
     progressCallback
   );
 
@@ -926,13 +926,13 @@ const exportEncryptDatabaseToFile = async ({
       progressCallback
     );
 
-    const result = await exportTable.export(accountId);
+    const result = await exportTable.export(accountId, accountEmail);
 
     exportProgress += 100 / PROGRESS_TOTAL_STEPS;
     handleProgressCallback(
       exportProgress,
       `saving_${exportTable.suffix}`,
-      `${recipientId}@${domain || APP_DOMAIN}`,
+      accountEmail,
       progressCallback
     );
 
@@ -943,7 +943,7 @@ const exportEncryptDatabaseToFile = async ({
   handleProgressCallback(
     exportProgress,
     'almost_done',
-    `${recipientId}@${domain || APP_DOMAIN}`,
+    accountEmail,
     progressCallback
   );
 };

electron_app/src/ipc/backup.js

@@ -22,6 +22,7 @@ const {
 const { APP_DOMAIN } = require('../utils/const');
 const { updateAccount } = require('./../database');
 const myAccount = require('../Account');
+const logger = require('../logger');
 let autoBackupsTime = [];
 let currentAutobackup = null;
 let nextBackupTimer = null;
@@ -68,6 +69,10 @@ const doExportBackupUnencrypted = async params => {
       ? accountObj.recipientId.split('@')
       : myAccount.recipientId.split('@');
 
+    logger.info(
+      `Unencrypted Backup Started For Account: ${recipientId}@${domain}`
+    );
+
     handleProgressCallback(
       -1,
       'starting_backup',
@@ -133,6 +138,7 @@ ipc.answerRenderer('export-backup-encrypted', async params => {
     await simulatePause(2000);
     globalManager.windowsEvents.enable();
     commitBackupStatus('local-backup-enable-events', 2);
+    logger.info(`Encrypted Backup Started for Path: ${backupPath}`);
     const backupSize = await exportBackupEncrypted({
       backupPath,
       password
@@ -222,7 +228,7 @@ const initAutoBackupMonitor = () => {
       !autoBackupNextDate ||
       account.id === currentAutobackup
     )
-      return;
+      continue;
 
     const now = moment();
     const pendingDate = moment(autoBackupNextDate);
@@ -235,6 +241,8 @@ const initAutoBackupMonitor = () => {
   }
 
   if (!currentAutobackup) checkNextBackup();
+
+  logger.debug(`Backups Started : ${JSON.stringify(autoBackupsTime)}`);
 };
 
 const checkNextBackup = () => {
@@ -246,24 +254,29 @@ const checkNextBackup = () => {
     return 0;
   });
 
+  logger.debug(
+    `Next Account : ${autoBackupsTime[0].accountId} : $ ${
+      autoBackupsTime[0].username
+    } in ${autoBackupsTime[0].triggerTimer} miliseconds`
+  );
   if (nextBackupTimer) clearTimeout(nextBackupTimer);
   nextBackupTimer = setTimeout(() => {
     initAutoBackup(autoBackupsTime[0].accountId);
   }, autoBackupsTime[0].triggerTimer);
 };
 
 const initAutoBackup = async accountId => {
+  currentAutobackup = accountId;
   autoBackupsTime = autoBackupsTime.filter(timer => {
     return timer.accountId !== accountId;
   });
-  currentAutobackup = accountId;
 
   const account = myAccount.loggedAccounts.find(acc => acc.id === accountId);
   if (!account) {
     backupDone();
     return;
   }
-
+  logger.info(`Backup Started For Account: ${account.recipientId}`);
   const {
     autoBackupEnable,
     autoBackupPath,
@@ -278,6 +291,7 @@ const initAutoBackup = async accountId => {
   try {
     const backupFileName = defineBackupFileName('db');
     const backupSize = await doExportBackupUnencrypted({
+      accountObj: account,
       backupPath: `${autoBackupPath}/${backupFileName}`,
       progressCallback: data => {
         send('backup-progress', data);
@@ -287,27 +301,28 @@ const initAutoBackup = async accountId => {
     const today = moment(Date.now());
     const nextDate = moment(autoBackupNextDate);
     do {
-      nextDate.add(1, timeUnit);
+      nextDate.add(1, 'hours');
     } while (nextDate.isBefore(today));
     await updateAccount({
       id: accountId,
       autoBackupLastDate: today.format(backupDateFormat),
       autoBackupLastSize: backupSize,
       autoBackupNextDate: nextDate.format(backupDateFormat)
     });
+    logger.debug(`Backups Finished : ${accountId}`);
     const timeDiff = nextDate.diff(today);
     autoBackupsTime.push({
       username: account.recipientId,
-      accountId,
+      id: accountId,
       triggerTimer: timeDiff <= 0 ? 1 : timeDiff
     });
 
     backupDone();
   } catch (backupErr) {
-    log(
+    logger.error(
       `Failed to do scheduled backup for account ${account.recipientId} : ${
         account.name
-      }`
+      } => ${backupErr}`
     );
     backupFail();
   }
@@ -316,7 +331,7 @@ const initAutoBackup = async accountId => {
 const backupDone = () => {
   currentAutobackup = null;
   nextBackupTimer = null;
-  checkNextBackup();
+  initAutoBackupMonitor();
 };
 
 const backupFail = () => {
@@ -340,15 +355,9 @@ ipc.answerRenderer('disable-auto-backup', accountId => {
     return timer.accountId !== accountId;
   });
 
-  if (!currentAutobackup) checkNextBackup();
+  if (!currentAutobackup) initAutoBackupMonitor();
 });
 
-const log = message => {
-  if (process.env.NODE_ENV === 'development') {
-    console.log(`[AutoBackup]: ${message}`);
-  }
-};
-
 process.on('exit', () => {
   clearTimeout(nextBackupTimer);
   currentAutobackup = null;

electron_app/src/windows/composer.js

@@ -91,6 +91,7 @@ const createComposerWindow = () => {
     showCopyImageAddress: false,
     prepend: (defaultActions, browserWindow) => {
       const checker = new spellChecker.Spellchecker();
+      checker.setSpellcheckerType(spellChecker.ALWAYS_USE_HUNSPELL);
 
       let options = [];
       if (defaultActions.misspelledWord) {

email_composer/src/utils/electronInterface.js

@@ -38,6 +38,7 @@ export const sendEventToMailbox = (name, params) => {
 webFrame.setSpellCheckProvider(mySettings.language, {
   spellCheck(words, callback) {
     const checker = new spellChecker.Spellchecker();
+    checker.setSpellcheckerType(spellChecker.ALWAYS_USE_HUNSPELL);
     const misspelled = words.filter(x => {
       return checker.isMisspelled(x);
     });

email_mailbox/src/components/PanelWrapper.js

@@ -270,7 +270,7 @@ class PanelWrapper extends Component {
   handleEnableAccountBackup = async backupPath => {
     const frequency = 'daily';
     const timeUnit = 'days';
-    const { nextDate } = getAutoBackupDates(Date.now(), 1, timeUnit);
+    const { nextDate } = getAutoBackupDates(Date.now(), 1, 'hours');
 
     await updateAccount({
       autoBackupEnable: true,

email_mailbox/src/components/SettingsAccountBackupWrapper.js

@@ -396,7 +396,7 @@ class SettingsAccountBackupWrapper extends Component {
   updateAutoBackupParams = async backupSize => {
     const { selectedFrequency, backupPath } = this.state;
     const timeUnit = defineUnitToAppend(selectedFrequency);
-    const { nowDate, nextDate } = getAutoBackupDates(Date.now(), 1, timeUnit);
+    const { nowDate, nextDate } = getAutoBackupDates(Date.now(), 1, 'hours');
     const autoBackupPath = removeFilenameFromPath(backupPath);
     await updateAccount({
       autoBackupEnable: true,