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

Full incognito support #119

Closed
robosharkler opened this issue Nov 19, 2022 · 4 comments
Closed

Full incognito support #119

robosharkler opened this issue Nov 19, 2022 · 4 comments

Comments

@robosharkler
Copy link

robosharkler commented Nov 19, 2022

I plan to add full incognito support to the browser, saving nothing at all for incognito mode.

The current design is saving temp history and clear when exit, though there might be case where app is not properly exited (power outage). Though before I start, want to know whether the current design is intended for some other reasons I might missed.

@truefedex
Copy link
Owner

Hi @shark-yn

Currently tvbro trying to isolate and remove any data stored by webview in incognito mode. Not only history.
See MainActivityViewModel.prepareSwitchToIncognito.

@robosharkler
Copy link
Author

Hi @truefedex ,

I am aware that tvbro will try to remove history in incognito mode at exit or toggle, though it still saves temporary history items into DB. I think incognito means we should avoid saving any history at first place, instead of saving temporary histories and removing them at exit (what if the app is not properly exited? The next person who opens the app might the see the incognito history). Though before I make the change, I want to know whether there is any other intention to save the temporary incognito history

@truefedex
Copy link
Owner

I agree with you @shark-yn - saving temporary browsing history in incognito mode can be turned off, as other browsers don't seem to save it even temporarily.

@truefedex
Copy link
Owner

Fixed in release (please reopen if not):
https://github.com/truefedex/tv-bro/releases/tag/v1.8.5

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

No branches or pull requests

2 participants