Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

track sanitizing migration slated for FF127/8 #1837

Closed
MagicalDrizzle opened this issue May 7, 2024 · 16 comments
Closed

track sanitizing migration slated for FF127/8 #1837

MagicalDrizzle opened this issue May 7, 2024 · 16 comments

Comments

@MagicalDrizzle
Copy link
Contributor

https://www.mozilla.org/en-US/firefox/126.0beta/releasenotes/

A simpler and more unified dialog for clearing user data is now available.
In addition to streamlining data categories, the new dialog also provides insights into the site data size corresponding to the selected time range.

pref("privacy.clearOnShutdown_v2.cache", true);
pref("privacy.clearOnShutdown_v2.cookiesAndStorage", true);
pref("privacy.clearOnShutdown_v2.historyFormDataAndDownloads", true);
pref("privacy.clearOnShutdown_v2.siteSettings", false);
pref("privacy.sanitize.clearOnShutdown.hasMigratedToNewPrefs", false);
pref("privacy.sanitize.cpd.hasMigratedToNewPrefs", false);

Maybe we should enforce the bottom two prefs as false to prevent data loss while this new thing is figured out?

@Thorin-Oakenpants
Copy link
Contributor

we have diffs issues, and I am aware of the new Data Sanitation and rollout - it's currently nightly only

sorry, just saw your last sentence after I labelled invalid .. still .. anyway, there will be no data loss - I mean it was just rolled back from beta due to exactly that (history was wiped when it shouldn't have)

also, pretty sure the migrated prefs will be changed at runtime, and setting them to false won't be effective, otherwise I think the migration will match exactly what you have in your overrides (super early nightly migration was hit and miss)

@Thorin-Oakenpants
Copy link
Contributor

^ in other words, 99% sure there's nothing for me to do until AFTER the rollout

@Thorin-Oakenpants
Copy link
Contributor

Thorin-Oakenpants commented May 7, 2024

here's the meta - https://bugzilla.mozilla.org/show_bug.cgi?id=1853996

and here's the new rollout - https://bugzilla.mozilla.org/show_bug.cgi?id=1895612

hope this lands in 128 so the next ESR is on the same playing field

Also, FYI, putting migrate prefs in the user.js is fraught (edit: with danger!!), because prefsCleaner would reset them and cause another migration (IIUIC)

@MagicalDrizzle
Copy link
Contributor Author

The new sanitation thing is more than "just" the GUI change right? because FF126 definitely has the new dialog thingy

@Thorin-Oakenpants Thorin-Oakenpants changed the title New sanitize prefs in FF126 beta track sanitizing migration slated for FF127/8 May 8, 2024
@Thorin-Oakenpants
Copy link
Contributor

nah, they rolled it back, as I said. If they haven't hooked up the new prefs, they can't really hook up the new UI .. and if they hook up the new UI they'll need the new prefs - so it's all about the migration .. but who knows, maybe they can toggle each on individually but there's no point in doing that

okie-dokie

but I've reopened so we have a issue to track it (but I doubt we'll need it)

@Thorin-Oakenpants
Copy link
Contributor

^ this is a profile where I have no user.js (so nothing being cleared/reset) but I did set it up to always sanitize everything on close - but I can't see that affected the migration, because they have to migrate everyone at some point

@Thorin-Oakenpants
Copy link
Contributor

Thorin-Oakenpants commented May 8, 2024

it's already tee'd up for 128: https://bugzilla.mozilla.org/show_bug.cgi?id=1895612#c1 (won't be pushed since we're in RC week)

edit: and barring any mishaps they could always backport it to 127

@MagicalDrizzle
Copy link
Contributor Author

MagicalDrizzle commented May 8, 2024

aah, I see
beta channel has release candidate shipped while aurora does not
yeah that's probably some super common knowledge that today I just learned lol, my apologies for bothering

@Thorin-Oakenpants
Copy link
Contributor

I have beta here as well, but I almost never use it because it hates running concurrently with any other firefox.exe

@Thorin-Oakenpants
Copy link
Contributor

https://bugzilla.mozilla.org/show_bug.cgi?id=1895612#c5

pref migrations:

privacy.clearOnShutdown.history --> privacy.clearOnShutdown_v2.historyFormDataAndDownloads (only the history value matters, downloads and formdata are ignored in this migration privacy.clearOnShutdown.downloads, privacy.clearOnShutdown.formdata )

privacy.clearOnShutdown.cookies --> privacy.clearOnShutdown_v2.cookiesAndStorage (only cookie value matters, active logins and offline website data are ignored in this migration)

privacy.clearOnShutdown.cache --> privacy.clearOnShutdown_v2.cache

privacy.clearOnShutdown.siteSettings --> privacy.clearOnShutdown_v2.siteSettings

@Thorin-Oakenpants
Copy link
Contributor

so this means e.g. v2.historyFormDataAndDownloads is all in one and you can't choose history separate to forms+downloads (which were linked in the past) .. and vs2.cookiesAndStorage is a catchall for offline website data (IDB, localStorage etc) and active logins (which is old timey crap not logins per se)

nice, four options instead of 7- I think I can add this ahead of time without too messy

@Thorin-Oakenpants
Copy link
Contributor

Thorin-Oakenpants commented May 31, 2024

and that should just leave cpd which I think is all the new prefs in 125 - I'll deal with them there

@Thorin-Oakenpants
Copy link
Contributor

closing - this wasn't as messy as I thought - the 4 _v2 prefs slotted in nicely - thanks @MagicalDrizzle , this did need it's own special issue since the prefs are from old timey diffs and stuff

PS: adding the migrated prefs would mean they would get reset with prefsCleaner and cause repeat migrations which can get messy for users since it would happen after user.js (I think)

@Thorin-Oakenpants
Copy link
Contributor

there's also a privacy.clearOnShutdown_v2.downloads pref but it's hidden pref and legacy nonsense from the migration - since downloads was always controlled by history in v1, and is now tied into historyFormDataAndDownloads in v2, so I'm going to ignore it

@theltalpha
Copy link

https://bugzilla.mozilla.org/show_bug.cgi?id=1895612#c5

pref migrations:
privacy.clearOnShutdown.history --> privacy.clearOnShutdown_v2.historyFormDataAndDownloads (only the history value matters, downloads and formdata are ignored in this migration privacy.clearOnShutdown.downloads, privacy.clearOnShutdown.formdata )
privacy.clearOnShutdown.cookies --> privacy.clearOnShutdown_v2.cookiesAndStorage (only cookie value matters, active logins and offline website data are ignored in this migration)
privacy.clearOnShutdown.cache --> privacy.clearOnShutdown_v2.cache
privacy.clearOnShutdown.siteSettings --> privacy.clearOnShutdown_v2.siteSettings

What I have not understand: Are the legacy sanitizing prefs literally deprecated as of FF128 (ESR)? That means, as soon as we have the "v2" sanitizing prefs present in our user.js, may we at the same time remove the legacy sanitizing prefs? Or still in other words, what is the purpose of having both the legacy and the "v2" prefs in our user.js?

@Thorin-Oakenpants
Copy link
Contributor

There's a migration pref, so I would assume that once the migration is done, the old prefs are no longer used to set the new prefs

We have all the prefs in the user.js for legacy users - and they all match (i.e the legacy prefs match the new prefs). I'm not sure when the migration itself happens - before or after user.js kicks in. Worst case scenario it takes another FF start. But as I said, the user.js contains all of them and they match.

There is no reason to modify the user.js - when they are deprecated then they will be moved to the deprecated section for prefsCleaner and eventually removed

As for any overrides, personally, I would just keep them there until they are actually deprecated - but if you want, remove the old legacy overrides and try it out :) You should be fine

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants