librewolf/settings
10
29
Fork
You've already forked settings
31

Delete history on shutdown by default #105

Closed
Spearmint wants to merge 1 commit from Spearmint/settings:cleanhistoryonshutdown into master
pull from: Spearmint/settings:cleanhistoryonshutdown
merge into: librewolf:master
librewolf:master
librewolf:canary
librewolf:user
librewolf:getbrowserinfo
librewolf:rs-blocker
librewolf:dom_push
librewolf:set_to_version_8_4
librewolf:newtabpage_weatherinfo
First-time contributor
Copy link

Updates the default settings to delete history on shutdown. This matches
what's described in the comment directly above these settings and in the
librewolf documentation.

The change to not delete history by default was originally introduced in
48fa54f53a
with further changes in

as a response to librewolf/issues#2097

In that issue some users reported not being able to disable the cleanup
of history and as a fix for those users the default was changed to
disable the cleanup.

From what I can tell the original issue was caused by changes to the
preferences in the upstream, whereby the existing preferences were no
longer respected after the introduction of new preferences, thereby
causing the changes to seemingly revert for people who had configured
history to not be deleted. In any case these settings are respected
between restarts in the current version.

After that there were reports that these settings were not saved between
updates in the other direction:

I believe this is because the defaultPref changes to these settings
above flipped the settings for those users. This is because if a setting
is set to the current default, even if done so intentionally, it will
not be considered "modified" and so a change to the defaultPref will
override the intended setting. Specifically: Flipping a boolean
defaultPref will change that setting for everybody except those who have
it explicitly set using a pref override.

I can sympathize with people who had history they wanted to save
deleted, but I'm not sure that unexpectedly saving history for everybody
else is the right solution. Librewolf's documentation states that data
is deleted on close multiple times, and the comment immediately above
these settings says the same thing. I think it goes against the privacy
promises given to have the browser save data that people would
reasonably expect to have deleted. I don't think a browser that saves
all browsing history (and more) by default can reasonably call itself
focused on privacy.

Upstream firefox has a habit of refactoring preferences in a way that
changes the behavior for people who have non-default settings. .I've
experienced that very issue several times myself, for example I've had
to re-disable autoplay 3-4 times over the past 10 or years. I think
some way of writing preference translation tables that would set the
default value of new preferences based on the value of old preferences
would be an excellent feature to have introduced, but as far as I know
nothing like that is currently implemented.

Care is needed in reverting this change, as people who want history
saved may unexpectedly find it deleted yet again. This is ultimately a
limitation of the design of the preferences system, introducing a third
state to differentiate between unmodified and explicitly set to
default
would be needed to overcome this issue. Absent that I think the
best solution would be to give users warning that this will be changed,
so that people who want to keep it set to false can do so with a pref
override.

Updates the default settings to delete history on shutdown. This matches what's described in the comment directly above these settings and in the librewolf documentation. The change to not delete history by default was originally introduced in https://codeberg.org/librewolf/settings/commit/48fa54f53a0a1483e15c8a14ba284bb92dbbeee1 with further changes in * https://codeberg.org/librewolf/settings/commit/b85eaae0038eab71ddfa69f6d95e44d934ae2eff * https://codeberg.org/librewolf/settings/commit/79c96b762a4834922df3943eb5dc45c2e8942055 as a response to https://codeberg.org/librewolf/issues/issues/2097 In that issue some users reported not being able to disable the cleanup of history and as a fix for those users the default was changed to disable the cleanup. From what I can tell the original issue was caused by changes to the preferences in the upstream, whereby the existing preferences were no longer respected after the introduction of new preferences, thereby causing the changes to seemingly revert for people who had configured history to not be deleted. In any case these settings are respected between restarts in the current version. After that there were reports that these settings were not saved between updates in the other direction: * https://codeberg.org/librewolf/issues/issues/2711 * https://codeberg.org/librewolf/issues/issues/2588 I believe this is because the defaultPref changes to these settings above flipped the settings for those users. This is because if a setting is set to the current default, even if done so intentionally, it will not be considered "modified" and so a change to the defaultPref will override the intended setting. Specifically: Flipping a boolean defaultPref will change that setting for everybody except those who have it explicitly set using a `pref` override. I can sympathize with people who had history they wanted to save deleted, but I'm not sure that unexpectedly saving history for everybody else is the right solution. Librewolf's documentation states that data is deleted on close multiple times, and the comment immediately above these settings says the same thing. I think it goes against the privacy promises given to have the browser save data that people would reasonably expect to have deleted. I don't think a browser that saves all browsing history (and more) by default can reasonably call itself focused on privacy. Upstream firefox has a habit of refactoring preferences in a way that changes the behavior for people who have non-default settings. .I've experienced that very issue several times myself, for example I've had to re-disable autoplay 3-4 times over the past 10 or years. I think some way of writing preference translation tables that would set the default value of new preferences based on the value of old preferences would be an excellent feature to have introduced, but as far as I know nothing like that is currently implemented. Care is needed in reverting this change, as people who want history saved may unexpectedly find it deleted yet again. This is ultimately a limitation of the design of the preferences system, introducing a third state to differentiate between _unmodified_ and _explicitly set to default_ would be needed to overcome this issue. Absent that I think the best solution would be to give users warning that this will be changed, so that people who want to keep it set to false can do so with a `pref` override.
Updates the default settings to delete history on shutdown. This matches
what's described in the comment directly above these settings and in the
librewolf documentation.
The change to not delete history by default was originally introduced in
48fa54f53a
with further changes in
* b85eaae003
* 79c96b762a
as a response to librewolf/issues#2097
In that issue some users reported not being able to disable the cleanup
of history and as a fix for those users the default was changed to
disable the cleanup.
From what I can tell the original issue was caused by changes to the
preferences in the upstream, whereby the existing preferences were no
longer respected after the introduction of new preferences, thereby
causing the changes to seemingly revert for people who had configured
history to not be deleted. In any case these settings are respected
between restarts in the current version.
After that there were reports that these settings were not saved between
updates in the other direction:
* librewolf/issues#2711
* librewolf/issues#2588
I believe this is because the defaultPref changes to these settings
above flipped the settings for those users. This is because if a setting
is set to the current default, even if done so intentionally, it will
not be considered "modified" and so a change to the defaultPref will
override the intended setting. Specifically: Flipping a boolean
defaultPref will change that setting for everybody except those who have
it explicitly set using a `pref` override.
I can sympathize with people who had history they wanted to save
deleted, but I'm not sure that unexpectedly saving history for everybody
else is the right solution. Librewolf's documentation states that data
is deleted on close multiple times, and the comment immediately above
these settings says the same thing. I think it goes against the privacy
promises given to have the browser save data that people would
reasonably expect to have deleted. I don't think a browser that saves
all browsing history (and more) by default can reasonably call itself
focused on privacy.
Upstream firefox has a habit of refactoring preferences in a way that
changes the behavior for people who have non-default settings. .I've
experienced that very issue several times myself, for example I've had
to re-disable autoplay 3-4 times over the past 10 or years. I think
some way of writing preference translation tables that would set the
default value of new preferences based on the value of old preferences
would be an excellent feature to have introduced, but as far as I know
nothing like that is currently implemented.
Care is needed in reverting this change, as people who want history
saved may unexpectedly find it deleted yet again. This is ultimately a
limitation of the design of the preferences system, introducing a third
state to differentiate between _unmodified_ and _explicitly set to
default_ would be needed to overcome this issue. Absent that I think the
best solution would be to give users warning that this will be changed,
so that people who want to keep it set to false can do so with a `pref`
override.

privacy.clearOnShutdown_v2.historyFormDataAndDownloads does nothing anymore.

I have no idea about the history of breakage caused by us changing these prefs since this was before I got involved.

I would not agree with saying that we cannot provide privacy while keeping history and downloads saved. For what LibreWolf can realistically target/provide, saving the history and downloads is more in line. LibreWolf's goal is to improve privacy against tracking you encounter on websites, not from your own computer. If you cannot trust your own computer, deleting history and downloads will not improve the situation.

This is just my stance on this. The comment/docs are probably outdated and the current stance needs to be put there instead.

`privacy.clearOnShutdown_v2.historyFormDataAndDownloads` does nothing anymore. I have no idea about the history of breakage caused by us changing these prefs since this was before I got involved. I would not agree with saying that we cannot provide privacy while keeping history and downloads saved. For what LibreWolf can realistically target/provide, saving the history and downloads is more in line. LibreWolf's goal is to improve privacy against tracking you encounter on websites, not from your own computer. If you cannot trust your own computer, deleting history and downloads will not improve the situation. This is just my stance on this. The comment/docs are probably outdated and the current stance needs to be put there instead.

We also had a little discussion about this today, and the general consence so far pretty much is that we would like to keep the current behavior as a default.

As soon as I have some spare time, I will make a PR correcting the information on the website.

We also had a little discussion about this today, and the general consence so far pretty much is that we would like to keep the current behavior as a default. As soon as I have some spare time, I will make a PR correcting the information on the website.
Author
First-time contributor
Copy link

Thanks for taking the time to respond. I didn't mean to impose my own ideals on a project I'm not involved in. If saving the history is the intended/desired behavior independently of 2097 then documenting that works for me.

Thanks for taking the time to respond. I didn't mean to impose my own ideals on a project I'm not involved in. If saving the history is the intended/desired behavior independently of 2097 then documenting that works for me.
Spearmint closed this pull request 2026年01月19日 20:13:37 +01:00

@Spearmint Please don't get me wrong. Didn't thought you impose your ideals on it or anything like this.
This PR was great and I really really appreciate the effort you put into this. Thanks a lot for trying to help and improve LibreWolf <3

Its just that this is "broken" since so long, that it was time to check what goal we actually have, what the desired behavior should be.

I really hope to see you around again <3

@Spearmint Please don't get me wrong. Didn't thought you impose your ideals on it or anything like this. This PR was great and I really really appreciate the effort you put into this. Thanks a lot for trying to help and improve LibreWolf <3 Its just that this is "broken" since so long, that it was time to check what goal we actually have, what the desired behavior should be. I really hope to see you around again <3

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
librewolf/settings!105
Reference in a new issue
librewolf/settings
No description provided.
Delete branch "Spearmint/settings:cleanhistoryonshutdown"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?