Merge branch 'develop'

Author: marcelklehr

CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## [5.5.3] - 2025-04-27
+
+### Fixed
+* perf(GoogleDrive): Speed up "no changes" code path
+* perf(Folder#clone): Make Folder#clone use prototype inheritance to save ALOT of memory
+* perf(BrowserAccount): Don't use browser.bookmarks.getTree() if avoidable
+* refactor(isUsingBrowserTabs): Expose Resource#isUsingBrowserTabs
+* fix(CachingTreeWrapper): Allow changes to the live tree without disturbing the cache
+* fix(App#openInNewTab): Use browser.tabs to open new tab instead of window.open (Didn't work in Edge for Android)
+* fix(Default#applyAdditionFailsafe): Only kick in if at least 20 bookmarks are being added
+* fix(WebDAV): Add Depth header to PROPFIND for getting filesize
+
 ## [5.5.2] - 2025-04-16
 
 ### Fixed

_locales/fr/messages.json

@@ -84,7 +84,7 @@
     "message": "E028: Impossible de s'authentifier sur le serveur."
   },
   "Error029": {
-    "message": "E029: Sécurité: La synchronisation en cours aurait effacé {0}% de vos signets. La synchronisation a été annulée. Désactivez la Sécurité dans les paramètres de votre compte si vous souhaitez tout de même synchroniser."
+    "message": "E029: Sécurité: La synchronisation en cours aurait effacé {0}% de vos signets. La synchronisation a été annulée. Désactivez la sécurité dans les paramètres de votre compte si vous souhaitez tout de même synchroniser."
   },
   "Error030": {
     "message": "E030: Le déchiffrement de votre fichier de signets a échoué. Le mot de passe est peut être erronée ou le fichier corrompu."
@@ -119,6 +119,15 @@
   "Error040": {
     "message": "E040: Impossible de rechercher ce nom de fichier sur votre Google Drive"
   },
+  "Error041": {
+    "message": "E041: La taille du fichier de signets distant est différente de celle obtenue depuis le serveur. C'est peut-être dû à une erreur réseau. Si cette erreur persiste, contactez l'administrateur du serveur."
+  },
+  "Error042": {
+    "message": "E042: La taille du fichier de signets distant n'a pu être obtenue. Il est impossible de vérifier que le fichier de signet a été complètement téléchargé. Si cette erreur persiste, contactez l'administrateur du serveur."
+  },
+  "Error043": {
+    "message": "E043: Sécurité: La synchronisation en cours aurait ajouté {0}% à la quantité de vos signets. La synchronisation a été annulée. Désactivez la sécurité dans les paramètres de votre compte si vous souhaitez tout de même synchroniser."
+  },
   "LabelWebdavurl": {
     "message": "URL WebDAV"
   },
@@ -303,7 +312,7 @@
     "message": "Journaux de débogage"
   },
   "LabelFunddevelopment": {
-    "message": "Soutenir le développement"
+    "message": "💸 Soutenir le développement"
   },
   "DescriptionFunddevelopment": {
     "message": "Le développement de floccus est possible grâce à un modèle d'abonnement volontaire. Si vous pensez que mon travail est utile, et s'il vous est possible d'y investir un montant même minime, merci de le soutenir. De plus, prenez aussi le temps d'aller noter floccus sur le Web Store de votre choix.\nMerci ! 💙"
@@ -505,22 +514,22 @@
     "message": "Sécurité"
   },
   "DescriptionFailsafe": {
-    "message": "Parfois, des erreurs de configuration ou un bogue logiciel peuvent causer la suppression involontaire de données. Ces données deviennent alors irrécupérables. Pour empêcher cela, floccus n'effacera pas d'un seul coup plus de 50% de vos signets, à moins que vous ne désactiviez la Sécurité ici."
+    "message": "Parfois, des erreurs de configuration ou des bogues logiciel peuvent causer la suppression involontaire de données, qui deviennent alors irrécupérables, ou la duplication involontaire de données, ce qui crée des conflits difficiles à résoudre. Pour empêcher cela, floccus n'effacera pas d'un seul coup plus de 20% de vos signets, ni n'ajoutera plus de 20% de la quantité de vos signet d'un seul coup, à moins que vous ne désactiviez la sécurité ici."
   },
   "LabelFailsafeon": {
-    "message": "Activée. N'effacera pas plus de 50% de vos signets locaux sans vous en demander d'abord l'autorisation. (Recommandé)"
+    "message": "Activée. N'enlèvera ou n'ajoutera pas plus 20% de/à la quantité de vos signets locaux sans vous en demander d'abord l'autorisation. (Recommandé)"
   },
   "LabelFailsafeoff": {
-    "message": "Désactivée. Permet la suppression de plus de 50% de vos signets locaux sans vous en demander l'autorisation."
+    "message": "Désactivée. Permet la suppression ou l'ajout de plus de 20% à la quantité de vos signets locaux sans vous en demander d'abord l'autorisation."
   },
   "StatusFailsafeoff": {
-    "message": "Sécurité désactivée. Il y a un risque de perte de données involontaire. Il est recommandé de laisser active la Protection de secours dans les paramètres de votre profil."
+    "message": "Sécurité désactivée. Il y a un risque de perte ou de duplication de données involontaire. Il est recommandé d'activer la sécurité dans les paramètres de votre profil."
   },
   "LabelAdaptergoogledrive": {
     "message": "Google Drive"
   },
   "DescriptionAdaptergoogledrive": {
-    "message": "Synchronise vos signets à l'aide d'un fichier chiffré stocké sur votre Google Drive. Les signets HTTP, FTP, data, fichier ou JavaScript peuvent être synchronisés. Cette option supporte le chiffrement point à point."
+    "message": "Synchronise vos signets à l'aide d'un fichier (potentiellement chiffré) stocké sur votre Google Drive. Les signets HTTP, FTP, data, fichier ou JavaScript peuvent être synchronisés. Cette option supporte le chiffrement point à point."
   },
   "LabelLogingoogle": {
     "message": "Se connecter avec Google"
@@ -673,7 +682,7 @@
     "message": "Profil créé"
   },
   "DescriptionAccountcreated": {
-    "message": "Votre profil a été créé. Vous pouvez à présent fermer cet onglet."
+    "message": "Votre profil a été créé. Vous pouvez à présent fermer cet onglet. Avertissement: synchronisation et sauvegarde sont des opérations différentes. Assurez-vous d'effectuer régulièrement des sauvegardes de vos signets."
   },
   "DescriptionNonhttps": {
     "message": "Vous accédez à un serveur qui utilise un protocole non sécurisé. Il est recommandé de n'utiliser que des serveurs compatibles avec HTTPS"
@@ -709,7 +718,7 @@
     "message": "Profil(s) importé(s) avec succès."
   },
   "DescriptionSyncinprogress": {
-    "message": "Synchronisation en cours"
+    "message": "Synchronisation en cours."
   },
   "DescriptionSyncscheduled": {
     "message": "Ce profil sera bientôt synchronisé. Nous attendons que vos autres appareils ou vos autres profils sur cet appareil aient terminé leur synchronisation."
@@ -718,7 +727,7 @@
     "message": "Git via HTTPS"
   },
   "DescriptionAdaptergit": {
-    "message": "Synchronisez vos signets en les conservant dans un fichier situé sur un dépôt Git. Il n'y a pas d'interface Web accompagnant cette option, et vous pouvez l'utiliser avec n'importe quel serveur Git, tel que Github, Gitlab, Gitea, etc. Elle permet de synchroniser des signets HTTP, FTP, données, fichier et JavaScript. Cette option ne supporte pas le chiffrement point à point."
+    "message": "L'option Git synchronise vos signets en les conservant dans un fichier situé sur un dépôt Git. Il n'y a pas d'interface web accompagnant cette option, mais vous pouvez l'utiliser avec n'importe quel serveur Git, tel que Github, Gitlab, Gitea, etc. Elle permet de synchroniser des signets HTTP, FTP, données, fichier et JavaScript. Cette option ne supporte pas le chiffrement point à point. Cette option n'est actuellement pas disponible pour l'application mobile."
   },
   "LabelGiturl": {
     "message": "URL du dépôt utilisant HTTP"
@@ -787,7 +796,7 @@
     "message": "Ce signet existe déjà dans le profil sélectionné"
   },
   "LabelReportproblem": {
-    "message": "Rapporter ce probleme"
+    "message": "Signaler ce problème"
   },
   "DescriptionReportproblem": {
     "message": "Si vous désirez contacter directement les développeurs pour discuter d'un problème, vous pouvez le faire ici:"
@@ -818,5 +827,8 @@
   },
   "DescriptionAutosync": {
     "message": "Activer la synchronisation automatique déclenchera automatiquement une synchronisation, à un intervalle de temps configurable, ainsi qu'à la demande, quelques secondes après chaque changement. Si vous n'activez pas la synchronisation automatique, vous devrez la déclencher manuellement en cliquant sur le bouton de synchronisation."
+  },
+  "LabelOpeninnewtab": {
+    "message": "Ouvrir cette vue dans un nouvel onglet"
   }
 }

android/app/build.gradle

@@ -7,8 +7,8 @@ android {
         applicationId "org.handmadeideas.floccus"
         minSdkVersion rootProject.ext.minSdkVersion
         targetSdkVersion rootProject.ext.targetSdkVersion
-        versionCode 5005002
-        versionName "5.5.2"
+        versionCode 5005003
+        versionName "5.5.3"
         testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
         aaptOptions {
              // Files and dirs to omit from the packaged assets dir, modified to accommodate modern web apps.

manifest.chrome.json

@@ -2,7 +2,7 @@
   "manifest_version": 3,
   "name": "floccus bookmarks sync",
   "short_name": "floccus",
-  "version": "5.5.2",
+  "version": "5.5.3",
   "description": "__MSG_DescriptionExtension__",
   "icons": {
     "48": "icons/logo.png",

manifest.firefox.json

@@ -2,7 +2,7 @@
   "manifest_version": 2,
   "name": "floccus bookmarks sync",
   "short_name": "floccus",
-  "version": "5.5.2",
+  "version": "5.5.3",
   "description": "__MSG_DescriptionExtension__",
   "icons": {
     "48": "icons/logo.png",

package.json

@@ -1,6 +1,6 @@
 {
   "name": "floccus",
-  "version": "5.5.2",
+  "version": "5.5.3",
   "description": "Sync your bookmarks privately across browsers and devices",
   "scripts": {
     "build": "NODE_OPTIONS=--max-old-space-size=5000 gulp",

src/lib/Account.ts

@@ -14,6 +14,7 @@ import { isTest } from './isTest'
 import CachingAdapter from './adapters/Caching'
 import * as Sentry from '@sentry/vue'
 import AsyncLock from 'async-lock'
+import CachingTreeWrapper from './CachingTreeWrapper'
 
 declare const DEBUG: boolean
 
@@ -163,7 +164,7 @@ export default class Account {
     try {
       if (this.getData().syncing || this.syncing) return
 
-      const localResource = await this.getResource()
+      const localResource = new CachingTreeWrapper(await this.getResource())
       if (!(await this.server.isAvailable()) || !(await localResource.isAvailable())) return
 
       Logger.log('Starting sync process for account ' + this.getLabel())
@@ -283,9 +284,8 @@ export default class Account {
       await this.setData({ scheduled: false, syncing: 1 })
 
       // update cache
-      const cache = (await localResource.getBookmarksTree()).clone(false)
+      const cache = (await localResource.getCacheTree()).clone(false)
       this.syncProcess.filterOutUnacceptedBookmarks(cache)
-      this.syncProcess.filterOutUnmappedItems(cache, await mappings.getSnapshot())
       await this.storage.setCache(cache)
 
       if (this.server.onSyncComplete) {

src/lib/CacheTree.ts

@@ -0,0 +1,26 @@
+import CachingAdapter from './adapters/Caching'
+import { IResource } from './interfaces/Resource'
+import { Folder, ItemLocation, TItemLocation } from './Tree'
+
+export default class CacheTree extends CachingAdapter implements IResource<typeof ItemLocation.LOCAL> {
+  protected location: TItemLocation = ItemLocation.LOCAL
+
+  constructor() {
+    super({})
+  }
+
+  public setTree(tree: Folder<typeof ItemLocation.LOCAL>) {
+    this.bookmarksCache = tree.clone(false)
+    this.bookmarksCache.createIndex()
+  }
+
+  async getBookmarksTree(): Promise<Folder<typeof ItemLocation.LOCAL>> {
+    const tree = await super.getBookmarksTree()
+    tree.createIndex()
+    return tree as Folder<typeof ItemLocation.LOCAL>
+  }
+
+  isAvailable(): Promise<boolean> {
+    return Promise.resolve(true)
+  }
+}

src/lib/CachingTreeWrapper.ts

@@ -0,0 +1,77 @@
+import { CachingResource, OrderFolderResource } from './interfaces/Resource'
+import { Bookmark, Folder, ItemLocation } from './Tree'
+import CacheTree from './CacheTree'
+import Ordering from './interfaces/Ordering'
+
+export default class CachingTreeWrapper implements OrderFolderResource<typeof ItemLocation.LOCAL>, CachingResource<typeof ItemLocation.LOCAL> {
+  private innerTree: OrderFolderResource<typeof ItemLocation.LOCAL>
+  private cacheTree: CacheTree
+
+  constructor(innerTree: OrderFolderResource<typeof ItemLocation.LOCAL>) {
+    this.innerTree = innerTree
+    this.cacheTree = new CacheTree()
+  }
+
+  async getBookmarksTree(): Promise<Folder<typeof ItemLocation.LOCAL>> {
+    const tree = await this.innerTree.getBookmarksTree()
+    this.cacheTree.setTree(tree)
+    return tree
+  }
+
+  async createBookmark(bookmark:Bookmark<typeof ItemLocation.LOCAL>): Promise<string|number> {
+    const id = await this.innerTree.createBookmark(bookmark)
+    const cacheId = await this.cacheTree.createBookmark(bookmark.copy(false))
+    const cacheBookmark = this.cacheTree.bookmarksCache.findBookmark(cacheId)
+    cacheBookmark.id = id
+    cacheBookmark.parentId = bookmark.parentId
+    this.cacheTree.bookmarksCache.createIndex()
+    return id
+  }
+
+  async updateBookmark(bookmark:Bookmark<typeof ItemLocation.LOCAL>):Promise<void> {
+    await this.innerTree.updateBookmark(bookmark)
+    await this.cacheTree.updateBookmark(bookmark.copy(false))
+  }
+
+  async removeBookmark(bookmark:Bookmark<typeof ItemLocation.LOCAL>): Promise<void> {
+    await this.innerTree.removeBookmark(bookmark)
+    await this.cacheTree.removeBookmark(bookmark)
+  }
+
+  async createFolder(folder:Folder<typeof ItemLocation.LOCAL>): Promise<string|number> {
+    const id = await this.innerTree.createFolder(folder)
+    const cacheId = await this.cacheTree.createFolder(folder.copy(false))
+    const cacheFolder = this.cacheTree.bookmarksCache.findFolder(cacheId)
+    cacheFolder.id = id
+    cacheFolder.parentId = folder.parentId
+    this.cacheTree.bookmarksCache.createIndex()
+    return id
+  }
+
+  async orderFolder(id:string|number, order:Ordering<typeof ItemLocation.LOCAL>): Promise<void> {
+    await this.innerTree.orderFolder(id, order)
+    await this.cacheTree.orderFolder(id, order)
+  }
+
+  async updateFolder(folder:Folder<typeof ItemLocation.LOCAL>): Promise<void> {
+    await this.innerTree.updateFolder(folder)
+    await this.cacheTree.updateFolder(folder.copy(false))
+  }
+
+  async removeFolder(folder:Folder<typeof ItemLocation.LOCAL>): Promise<void> {
+    await this.innerTree.removeFolder(folder)
+    await this.cacheTree.removeFolder(folder)
+  }
+
+  isAvailable(): Promise<boolean> {
+    return this.innerTree.isAvailable()
+  }
+
+  async isUsingBrowserTabs() {
+    return this.innerTree.isUsingBrowserTabs?.()
+  }
+
+  getCacheTree(): Promise<Folder<typeof ItemLocation.LOCAL>> {
+    return this.cacheTree.getBookmarksTree()
+  }
+}

src/lib/Diff.ts

@@ -200,16 +200,16 @@ export default class Diff<L1 extends TItemLocation, L2 extends TItemLocation, A
           const newId = action.payload.id
           newAction = {
             ...action,
-            payload: action.payload.cloneWithLocation(false, targetLocation),
-            oldItem: action.oldItem.cloneWithLocation(false, action.payload.location)
+            payload: action.payload.copyWithLocation(false, targetLocation),
+            oldItem: action.oldItem.copyWithLocation(false, action.payload.location)
           }
           newAction.payload.id = oldId
           newAction.oldItem.id = newId
         } else {
           newAction = {
             ...action,
-            payload: action.payload.cloneWithLocation(false, targetLocation),
-            oldItem: action.payload.clone(false)
+            payload: action.payload.copyWithLocation(false, targetLocation),
+            oldItem: action.payload.copy(false)
           }
           newAction.payload.id = Mappings.mapId(mappingsSnapshot, action.payload, targetLocation)
         }
@@ -253,8 +253,8 @@ export default class Diff<L1 extends TItemLocation, L2 extends TItemLocation, A
     return this.getActions().map((action: A) => {
       return {
         ...action,
-        payload: action.payload.clone(false),
-        oldItem: action.oldItem && action.oldItem.clone(false),
+        payload: action.payload.copy(false),
+        oldItem: action.oldItem && action.oldItem.copy(false),
       }
     })
   }