-
Notifications
You must be signed in to change notification settings - Fork 14
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
Expand Fireproofing to include Local Storage and IndexedDB #408
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Didn't test on 12.2 but commented out the available check and it left indexed db database in place when burning.
@brindy So sorry – I realized belatedly that the additional step I added to remove Local Storage and IndexedDB for domains that aren't Fireproof hadn't been pushed yet (without that change, all domains would have had this data retained), so I've done that and added a bit of bonus variable/function renaming to clear things up. Would you be able to take another look? 🙏 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sense! LGTM!
# By Alexey Martemyanov (20) and others # Via Tomas Strba (2) and others * develop: (63 commits) Tweaks of suggestions and autocomplete (#403) Bump privacy dashboard to latest version (#409) Point to the latest BrowserServicesKit branch. (#414) Move embedded TDS from BSK to platform repo (#412) Image of shield with dot replaced (#410) Expand Fireproofing to include Local Storage and IndexedDB (#408) Version 0.18.5 support privacy config for clickToLoad (#407) Automatically select available login (#405) initial FB Click to Load (WIP) (#329) onboarding updates (#398) Version 0.18.4 Configuration of Sparkle - Setting SUAllowsAutomaticUpdates to NO (#404) Hide downloads button if the popover is opened/closed manually (#397) Textfield of the homepage is empty and unfocused right after switching to the homepage (#400) Remove navigatorCredentials (#392) Remove GPC header if it exists when not needed (#366) Version 0.18.3 Fireproofing encrypted storage (#332) Fix Lock Screen UI issues (#399) ... # Conflicts: # DuckDuckGo.xcodeproj/project.pbxproj # DuckDuckGo/Crash Reports/Model/CrashReportSender.swift
Task/Issue URL: https://app.asana.com/0/1177771139624306/1200042850887108/f
Tech Design URL:
CC:
Description:
This PR updates the Fireproofing feature to preserve Local Storage and IndexedDB. It only preserves IndexedDB on macOS 12.2+, as earlier versions have a privacy flaw related to that feature (see the comment in the code for more).
Steps to test this PR:
Testing Fireproof sites:
Testing non-Fireproof sites:
removeLocalStorageAndIndexedDBForNonFireproofDomains
Testing checklist:
Internal references:
Software Engineering Expectations
Technical Design Template
When ready for review, remember to post the PR in MM