Skip to content
This repository was archived by the owner on Dec 11, 2019. It is now read-only.

Clean private tab on tab close #6684

Closed
hmrodrigues opened this issue Jan 16, 2017 · 9 comments
Closed

Clean private tab on tab close #6684

hmrodrigues opened this issue Jan 16, 2017 · 9 comments
Assignees
Labels
feature/private-tabs fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. priority/P4 Minor loss of function. Workaround usually present. wontfix

Comments

@hmrodrigues
Copy link

Did you search for similar issues before submitting this one?
Yes

Describe the issue you encountered:
When you close the last incognito tab, the browsing data is stored until the browser is restarted

Expected behavior:
Clean incognito browsing data on last incognito tab closed

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Linux (Ubuntu 14.04) and Arch Linux
  • Brave Version (revision SHA):
    0.12.15
  • Steps to reproduce:
    1. Open Private Tab
    2. Open page that stores cookies (In my case Odoo that saves login and seleted database)
    3. Close Private Tab
    4. Reopen Private Tab and page.
    5. Page finds cookies and does autologin
@diracdeltas
Copy link
Member

diracdeltas commented Aug 31, 2017

2 options

  1. all private tabs that are open at the same time share the same session. when the last private tab is closed and then a new private tab is opened, the new private tab is in a new session. this would achieve @hmrodrigues's expected behavior.
  2. each private tab has its own session

i feel that #2 is more intuitive behavior and also easier to implement

@diracdeltas diracdeltas added this to the 0.21.x (Nightly Channel) milestone Aug 31, 2017
@diracdeltas
Copy link
Member

related: #597

@diracdeltas
Copy link
Member

diracdeltas commented Aug 31, 2017

@bbondy mentions the second option might be harder with tab opener relationships so we could try starting with option 1

@diracdeltas diracdeltas self-assigned this Aug 31, 2017
@SilverPuppy
Copy link

SilverPuppy commented Sep 1, 2017

Having private tabs spawned from links in other private tabs share a session with the tab which spawned them would be expected behavior to most users. Having each private tab use its own session regardless of its origin would be annoying because if someone were signed in in the originating tab, the newly-spawned tab would not be, which would at the least require another sign-in unnecessarily.

When a new private tab is opened manually, it should create a new session. It should share that session with all tabs descended from the original tab or its children. When the last tab in the private session is closed, that session should cease to exist. Other private tabs with different sessions should keep their own sessions alive until the last tab in the session is closed.

From a privacy standpoint, option 2 is definitely preferable to the current implementation, provided that it is amended slightly to read "each private tab has its own session WHICH IS DESTROYED WHEN THE TAB IS CLOSED."

Also, kudos for getting back to this.

@diracdeltas
Copy link
Member

@bbondy mentions the second option might be harder with tab opener relationships so we could try starting with option 1

oops, i originally switched options1 and 2 in this comment. it's been corrected.

diracdeltas added a commit that referenced this issue Sep 7, 2017
partial fix for #6684
and #8963

TODO: Either the private tab sessions should be cleared and revert back to private session 1 when the last private tab is closed, or we need a private tab session manager menu. Currently the session will keep incrementing until the browser restarts.

Test Plan:
1. open private tab
2. in the private tab, go to a site that requires login
3. login, then open a link from that site in a new private tab
4. you should also be logged in in the second private tab
5. open a new private tab
6. go to the same site. you should not be logged in.
diracdeltas added a commit that referenced this issue Sep 7, 2017
partial fix for #6684
and #8963

TODO: Either the private tab sessions should be cleared and revert back to private session 1 when the last private tab is closed, or we need a private tab session manager menu. Currently the session will keep incrementing until the browser restarts.

Test Plan:
1. open private tab
2. in the private tab, go to a site that requires login
3. login, then open a link from that site in a new private tab
4. you should also be logged in in the second private tab
5. open a new private tab
6. go to the same site. you should not be logged in.
diracdeltas added a commit that referenced this issue Sep 11, 2017
partial fix for #6684
and #8963

TODO: Either the private tab sessions should be cleared and revert back to private session 1 when the last private tab is closed, or we need a private tab session manager menu. Currently the session will keep incrementing until the browser restarts.

Test Plan:
1. open private tab
2. in the private tab, go to a site that requires login
3. login, then open a link from that site in a new private tab
4. you should also be logged in in the second private tab
5. open a new private tab
6. go to the same site. you should not be logged in.
@NumDeP
Copy link

NumDeP commented Sep 30, 2017

I'm curious, would the same set of rules apply to TPB as well? #1185

I don't mean to go off topic but I mention this specifically as I think it does sort of pertain to this issue, particularly your 2nd option of each private tab having it's own session that is if you do apply the same rules to the TPB tabs as well when it comes to it during or after 1.2. #1185
I'm basing this on the fact that users not wanting to make cookies and such traces available on the new (session) tab but then if the content of each tab is related to each other I don't see what the point of it would be.
Personally, I very rarely open more than a hand full of tor tabs if or whenever I do use it but don't you think this will just exacerbate it more in Brave?

It will be interesting to ascertain how you juxtapose the New Tor Circuit on top of the private tab or will the overall menu be changed to consolidate it?

I suspect having the same session (as well as sharing the same IPs) from the first tab on each New Tab after seems better unless a users selects New Private Session. At least from a privacy context it does seem better because having a different set of IPs for each New Tab depending on what a user may be doing could be damaging.

There was an interesting report made by a German duo where they analysed mechanisms and lengths to which industries go to de-anonymizing user data.
The researcher Dewes called it, “combinatorial deanonymization”, based on earlier research...published in 2007. This takes advantage of the fact that you only need around 10 URLs to identify someone uniquely – it turns out that people are very different in their browsing habits. So if there is additional information available that links data in those URLs to an identity, it is likely that all of the other URLs also relate to that person too.'
I can't speak too much about the validity of it, only I hope it isn't as accurate as it states though you're probably already aware of it.
The report is available on PIA's blog and here - https://community.brave.com/t/browser-extensions-disclose-personal-data/5305

Sorry for boring you; you're welcome to delete this comment if you don't think it aligns with the issue at hand. Thanks.

diracdeltas added a commit that referenced this issue Oct 11, 2017
partial fix for #6684
and #8963

TODO: Either the private tab sessions should be cleared and revert back to private session 1 when the last private tab is closed, or we need a private tab session manager menu. Currently the session will keep incrementing until the browser restarts.

Test Plan:
1. open private tab
2. in the private tab, go to a site that requires login
3. login, then open a link from that site in a new private tab
4. you should also be logged in in the second private tab
5. open a new private tab
6. go to the same site. you should not be logged in.
diracdeltas added a commit that referenced this issue Oct 25, 2017
partial fix for #6684
and #8963

TODO: Either the private tab sessions should be cleared and revert back to private session 1 when the last private tab is closed, or we need a private tab session manager menu. Currently the session will keep incrementing until the browser restarts.

Test Plan:
1. open private tab
2. in the private tab, go to a site that requires login
3. login, then open a link from that site in a new private tab
4. you should also be logged in in the second private tab
5. open a new private tab
6. go to the same site. you should not be logged in.
@bbondy bbondy modified the milestones: 0.21.x (Developer Channel), 0.20.x (Beta Channel), Backlog Oct 25, 2017
@alexwykoff alexwykoff added the priority/P4 Minor loss of function. Workaround usually present. label Oct 31, 2017
@bbondy bbondy modified the milestones: Triage Backlog, Prioritized Backlog Nov 2, 2017
@cezaraugusto cezaraugusto modified the milestones: Prioritized Backlog, Triage Backlog Nov 8, 2017
@bsclifton bsclifton modified the milestones: Triage Backlog, Prioritized Backlog Nov 21, 2017
diracdeltas added a commit that referenced this issue Dec 4, 2017
partial fix for #6684
and #8963

TODO: Either the private tab sessions should be cleared and revert back to private session 1 when the last private tab is closed, or we need a private tab session manager menu. Currently the session will keep incrementing until the browser restarts.

Test Plan:
1. open private tab
2. in the private tab, go to a site that requires login
3. login, then open a link from that site in a new private tab
4. you should also be logged in in the second private tab
5. open a new private tab
6. go to the same site. you should not be logged in.
@diracdeltas diracdeltas modified the milestones: Backlog (Prioritized), 0.20.x (Beta Channel) Dec 4, 2017
@bsclifton bsclifton modified the milestones: 0.20.x (Beta Channel), 0.21.x (Developer Channel) Dec 22, 2017
@bsclifton
Copy link
Member

Moving to 0.21.x since we aren't using Muon 4.6.x yet

@diracdeltas
Copy link
Member

diracdeltas commented Dec 26, 2017

@NumDeP Tor Private Tab will be a new tab type, independent of private/session tabs as they are currently implemented. There will be no such thing as 'TPB New Session Tab'; i imagine there will just be a menu item for 'New TPB Tab'.

My initial thinking on this was that TBP tabs would share the same session state but run in different Tor circuits (same to how tabs in Tor browser work right now). If a user wanted to clear all session state and use fresh circuits, there would be a 'New Identity' button.

diracdeltas added a commit that referenced this issue Jan 2, 2018
partial fix for #6684
and #8963

TODO: Either the private tab sessions should be cleared and revert back to private session 1 when the last private tab is closed, or we need a private tab session manager menu. Currently the session will keep incrementing until the browser restarts.

Test Plan:
1. open private tab
2. in the private tab, go to a site that requires login
3. login, then open a link from that site in a new private tab
4. you should also be logged in in the second private tab
5. open a new private tab
6. go to the same site. you should not be logged in.
@bsclifton bsclifton modified the milestones: 0.21.x (Beta Channel), 0.22.x (Developer Channel) Feb 15, 2018
@bbondy bbondy modified the milestones: 0.22.x (Developer Channel), 0.23.x (Nightly Channel) Feb 25, 2018
@bsclifton bsclifton modified the milestones: 0.23.x (Nightly Channel), Completed work Feb 28, 2018
@LaurenWags
Copy link
Member

+1 via email

@alexwykoff alexwykoff modified the milestones: Completed work, 0.24.x (Nightly Channel) Apr 10, 2018
@tildelowengrimm tildelowengrimm added the fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. label Apr 17, 2018
@srirambv srirambv removed this from the 0.24.x (Nightly Channel) milestone Apr 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature/private-tabs fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. priority/P4 Minor loss of function. Workaround usually present. wontfix
Projects
None yet
Development

Successfully merging a pull request may close this issue.