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

Data from normal tab is carried over into private tab #1102

Closed
srirambv opened this issue May 21, 2019 · 2 comments
Closed

Data from normal tab is carried over into private tab #1102

srirambv opened this issue May 21, 2019 · 2 comments

Comments

@srirambv
Copy link
Contributor

@srirambv srirambv commented May 21, 2019

Description:

Data from normal tab is carried over into private tab

Steps to Reproduce

  1. Visit jbhifi.com.au
  2. Add a product to shopping cart
  3. Long press on a different product and open in private tab via context menu
  4. Shopping cart in private tab already has the item added in step 2

Actual result:
Data from normal tab is carried over into private tab

Expected result:
No data should be carried over from normal tab to private tab

Reproduces how often: [Easily reproduced, Intermittent Issue]
Easy

Brave Version:
1.9.3(19.05.20.17)

Device details:
All

Website problems only:

  • did you check with Brave Shields down?
  • did you check in Safari/Firefox (WkWebView-based browsers)?

Additional Information

cc: @danishjafri88 @jumde

@danishjafri88
Copy link
Contributor

@danishjafri88 danishjafri88 commented May 21, 2019

A regression of #1100. In this case The privacy protection is determined from a global variable. This is used to set Data store on WebView init. The problem is for this particular and similar scenarios(Any way to switch to and fro the privacy mode), The tab is created before the variable is updated.

@LaurenWags
Copy link

@LaurenWags LaurenWags commented Jul 15, 2019

  • Verified passed on iPad 5th Gen (iOS 12.3.1) running 1.10 (19.07.12.14). Verified STR from description.
  • Verification PASSED on iPhone 6s+ running iOS 12.3.1 using 1.10 (19.07.16.23)
@kjozwiak kjozwiak added this to the 1.10 milestone Jul 19, 2019
@srirambv srirambv added Epic: Security and removed security labels Aug 29, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.