Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Consider Not Restoring Tabs when Disabling PBO #580

Closed
jhreis opened this issue Dec 5, 2018 · 7 comments · Fixed by #1382
Closed

Consider Not Restoring Tabs when Disabling PBO #580

jhreis opened this issue Dec 5, 2018 · 7 comments · Fixed by #1382

Comments

@jhreis
Copy link
Contributor

jhreis commented Dec 5, 2018

When a user turns on PBO on, we should prompt them that this will destroy all of their tabs (potentially other data?). This prevents a user who stays in PBO mode for many months from having super old tabs being restored. It can appear to be a security vulnerability.

Ref: #576

@jhreis
Copy link
Contributor Author

jhreis commented Apr 1, 2019

Need final copy on what this user-message/confirmation should be.

@jhreis jhreis added blocked: needs info Needs further information before work can commence and removed suggestion labels Apr 1, 2019
@anthonypkeane
Copy link

Text for alert:

Enabling Private Browsing Only will close your current tabs

Cancel | Okay

@tomlowenthal can you pls confirm?

@tildelowengrimm
Copy link

Private Browsing Only mode will close all your current tabs and log you out of all sites.

@jhreis
Copy link
Contributor Author

jhreis commented Aug 8, 2019

Do we have final product / security approval for this then @tomlowenthal and @anthonypkeane sounds like we do using the above copy?

@anthonypkeane
Copy link

anthonypkeane commented Aug 8, 2019

The alert text is

Private Browsing Only mode will close all your current tabs and log you out of all sites.
Cancel | Okay

@jhreis
Copy link
Contributor Author

jhreis commented Aug 9, 2019

Note to whoever picks this up. We then need to clear cookies to make sure users are also logged out of all of their sites.

@jhreis jhreis removed the blocked: needs info Needs further information before work can commence label Aug 9, 2019
@Brandon-T Brandon-T self-assigned this Aug 12, 2019
@anthonypkeane anthonypkeane added this to the 1.15 milestone Nov 6, 2019
@jhreis jhreis modified the milestones: 1.15, 1.14 Nov 14, 2019
@srirambv
Copy link
Contributor

Verification passed on iPhone XR with iOS 13.2 running 1.14(19.11.27.14)

  • Verified enabling PBO shows warn message
  • Verified logged in session in a normal tab is lost when PBO is enabled
  • Verified disabling PBO goes back to a normal tab and tabs from previous sessions are lost
  • Verified browsing history is not lost when PBO is toggled
  • Verified site shields settings are not lost when PBO is toggled

Verification passed on iPhone 7+ with iOS 13.2 running 1.14(19.11.27.14)

  • Verified enabling PBO shows warn message
  • Verified logged in session in a normal tab is lost when PBO is enabled
  • Verified disabling PBO goes back to a normal tab and tabs from previous sessions are lost
  • Verified browsing history is not lost when PBO is toggled
  • Verified site shields settings are not lost when PBO is toggled

Verification passed on iPhone 6 with iOS 12.4 running 1.14(19.11.27.14)

  • Verified enabling PBO shows warn message
  • Verified logged in session in a normal tab is lost when PBO is enabled
  • Verified disabling PBO goes back to a normal tab and tabs from previous sessions are lost
  • Verified browsing history is not lost when PBO is toggled
  • Verified site shields settings are not lost when PBO is toggled

Verification passed on iPad Pro with iOS 13.2 running 1.14(19.11.27.14)

  • Verified enabling PBO shows warn message
  • Verified logged in session in a normal tab is lost when PBO is enabled
  • Verified disabling PBO goes back to a normal tab and tabs from previous sessions are lost
  • Verified browsing history is not lost when PBO is toggled
  • Verified site shields settings are not lost when PBO is toggled

Verification passed on iPad Pro with iOS 12.4 running 1.14(19.11.27.14)

  • Verified enabling PBO shows warn message
  • Verified logged in session in a normal tab is lost when PBO is enabled
  • Verified disabling PBO goes back to a normal tab and tabs from previous sessions are lost
  • Verified browsing history is not lost when PBO is toggled
  • Verified site shields settings are not lost when PBO is toggled

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