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

Cross-site tests are not working as expected #11930

Closed
GeetaSarvadnya opened this issue Sep 30, 2020 · 9 comments
Closed

Cross-site tests are not working as expected #11930

GeetaSarvadnya opened this issue Sep 30, 2020 · 9 comments
Labels
closed/duplicate Issue has already been reported OS/Android Fixes related to Android browser functionality OS/Desktop privacy/feature User-facing privacy- & security-focused feature work. privacy-pod Feature work for the Privacy & Web Compatibility pod QA/Yes

Comments

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Sep 30, 2020

Description

Follow up of #9020
Cross-site tests are not working as expected

Steps to Reproduce

  1. Open https://fmarier.github.io/brave-testing/query-filter.html
  2. Perform cross-site tests in the Private window tab (close all private windows in between each test case)
  3. In each case the fbclid parameter is present on the landing page.

Actual result:

In each case the fbclid parameter is present on the landing page.
image

Expected result:

In all of these cases, the fbclid parameter should be missing from the landing page.

Reproduces how often:

Always

Desktop Brave version:

Brave 1.16.46 Chromium: 86.0.4240.55 (Official Build) nightly (64-bit)
Revision a6d625ef6f7fe8ea0675f1cf759155a05ee1be40-refs/branch-heads/4240@{#953}
OS Windows 10 OS Version 1903 (Build 18362.1016)

Android Device details:

  • Install type (ARM, x86):
  • Device type (Phone, Tablet, Phablet):
  • Android version:

Version/Channel Information:

  • Can you reproduce this issue with the current release? No
  • Can you reproduce this issue with the beta channel? No
  • Can you reproduce this issue with the nightly channel? Yes

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? NA
  • Does the issue resolve itself when disabling Brave Rewards? NA
  • Is the issue reproducible on the latest version of Chrome? NA

Miscellaneous Information:

cc: @brave/legacy_qa @fmarier @rebron

@GeetaSarvadnya GeetaSarvadnya added privacy/feature User-facing privacy- & security-focused feature work. QA/Yes OS/Android Fixes related to Android browser functionality privacy-pod Feature work for the Privacy & Web Compatibility pod OS/Desktop labels Sep 30, 2020
@btlechowski
Copy link

Likely dupe of #11924

@fmarier
Copy link
Member

fmarier commented Sep 30, 2020

I've just tested this on the following versions:

  • Version 1.17.3 Chromium: 86.0.4240.55 (Official Build) nightly (64-bit)
  • Version 1.16.46 Chromium: 86.0.4240.55 (Official Build) nightly (64-bit)
  • Version 1.15.67 Chromium: 86.0.4240.55 (Official Build) beta (64-bit)

and the cross-site tests work fine for me in all cases.

Is this Windows-specific? Anything else I should do to reproduce?

@LaurenWags
Copy link
Member

LaurenWags commented Oct 5, 2020

@fmarier reproduces for me using these steps:

  1. Clean profile with 1.16.x
  2. Open https://fmarier.github.io/brave-testing/query-filter.html in normal window
  3. Choose a link from Cross-site tests section and right click on it
  4. Choose "Open link in private window"
  5. Private window opens with link
  6. Look at URL bar --> tracking parameter is included

If I do the following, then the tracking parameter is not included:

  1. Clean profile with 1.16.x
  2. Open https://fmarier.github.io/brave-testing/query-filter.html in private window
  3. Choose a link from Cross-site tests section and click on it
  4. Look at URL bar --> tracking parameter is not included

or

  1. Clean profile with 1.16.x
  2. Open https://fmarier.github.io/brave-testing/query-filter.html in normal window
  3. Choose a link from Cross-site tests section and click on it
  4. Look at URL bar --> tracking parameter is not included
Brave 1.16.52 Chromium: 86.0.4240.68 (Official Build) dev (x86_64)
Revision ad72ee9aa8e15ed300df1238e76c7a8f4d686f97-refs/branch-heads/4240@{#1097}
OS macOS Version 10.14.6 (Build 18G3020)

cc @GeetaSarvadnya to provide additional info

@GeetaSarvadnya
Copy link
Author

GeetaSarvadnya commented Oct 5, 2020

cross site testing

@fmarier I have followed the same steps which are mentioned in the STR. The issue is reproducible please find the gif.
cc: @LaurenWags

@LaurenWags
Copy link
Member

@GeetaSarvadnya does it reproduce if you click on the link instead of copy/paste the link address?

@GeetaSarvadnya
Copy link
Author

@LaurenWags Good catch, If I directly click on the link it works fine, issues arise only when copying the link and paste it in a new private tab

@LaurenWags
Copy link
Member

thanks for checking @GeetaSarvadnya

@fmarier hopefully the above comments help, please let us know if we can provide additional information.

@fmarier
Copy link
Member

fmarier commented Oct 5, 2020

Good catch, If I directly click on the link it works fine, issues arise only when copying the link and paste it in a new private tab

Thanks @LaurenWags and @GeetaSarvadnya. I just realized how the instructions on my test page were misleading. I've clarified that the test page itself should be opened in a private window.

This is indeed a duplicate of #11924.

@LaurenWags
Copy link
Member

awesome, thanks @fmarier 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed/duplicate Issue has already been reported OS/Android Fixes related to Android browser functionality OS/Desktop privacy/feature User-facing privacy- & security-focused feature work. privacy-pod Feature work for the Privacy & Web Compatibility pod QA/Yes
Projects
None yet
Development

No branches or pull requests

4 participants