Disable bookmark backups (easy fix) (!)

Tor browser shouldn't backup bookmarks at all and even when bookmarks are deleted, old backups remain in:

/Browser/TorBrowser/Data/Browser/profile.default/bookmarkbackups

added component::applications/tor browser gitlab-tb-torbutton owner::tbb-team priority::high severity::normal status::new tbb-easy tbb-torbutton type::defect labels

Trac:
Component: - Select a component to Tor Browser
Owner: N/A to tbb-team

Wanted to create a ticket for this as well, bumping instead, deleted bookmarks should not be kept without the user knowing it is kept.

I often delete bookmarks I don't consider relevant anymore, but knowing they're being backed up in Browser/TorBrowser/Data/Browser/profile.default/bookmarkbackups forces me to check that folder every few days and purge it.

EDIT: Can be disabled by setting browser.bookmarks.max_backups to 0 in about:config

Trac:
Priority: Medium to High
Reviewer: N/A to N/A

Trac:
Summary: Disable bookmark backups to Disable bookmark backups (easy fix) (!)
Type: defect to enhancement

What? Deleted bookmarks remain stored in "bookmarkbackups", why?

I've changed "browser.bookmarks.max_backups" to "0" in "about:config" for myself, like the comment above me says.

Is there a reason that this has been ignored for over 7 months? I mean come on, please fix this soon!

Thanks =)

Trac:
Type: enhancement to defect

Binding this pref change to the option to disable Private Browsing Mode in Tor Browser (Don't record browsing history or website data (enabled Private Browsing Mode)) seems the way to go here. Patches are welcome!

Trac:
Keywords: N/A deleted, tbb-easy added

Trac:
Keywords: N/A deleted, tbb-torbutton added

I'm not convinced that disabling bookmark backups is a good idea, but it might make sense to reduce the number of backups that are kept by default. It is entirely possible that the SQLite database can get corrupted, and if you're not manually backing up your bookmarks, you'd lose of all of them.

Tying this to private browsing mode doesn't make any sense to me, because bookmarks are kept regardless of this setting.

Closed #32988 (moved) as dupe (?).

Hello everyone, just joined and would like to contribute to the project.

Is this defect still valid? It's pretty old but marked as an "easy fix" so I could start my journey with it.

I can see how up to 15 (default) days of daily bookmark backups could be an issue: assuming your device was compromised: i.e old bookmarks recently deleted would have a persistent disk write for the duration, but what if a user needs to restore them. I also think if your device was compromised, you would have more serious issues than this.

That said, I'm not totally against it: but maybe changing the default value to 2 or 3 might be a better compromise: at least a value of 1 gives a user a chance to instantly retrieve something

I haven't tested anything: so I'm not sure if changing the value removes old ones on a daily basis until it catches up, or if they persist

Trac:
Keywords: N/A deleted, gitlab-tb-torbutton added

mentioned in issue #32988 (moved)

moved to tpo/applications/torbutton#17584 (moved)