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

Tor window always starts with default shields setting #2978

Closed
simonhong opened this issue Jan 16, 2019 · 3 comments
Closed

Tor window always starts with default shields setting #2978

simonhong opened this issue Jan 16, 2019 · 3 comments
Labels
feature/shields The overall Shields feature in Brave. feature/tor/guest-semantics Exposing the fact that private windows with Tor are built on guest sessions. feature/tor priority/P4 Planned work. We expect to get to it "soon". QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Yes release-notes/include

Comments

@simonhong
Copy link
Member

Description

Tor window always starts with default shields settings when user has different shields settings in normal profile

Steps to Reproduce

  1. Change any settings in default shields settings in normal window
  2. Launch tor window
  3. Check it is applied to tor window

Actual result:

Tor window always starts with default shields setting

Expected result:

Tor window starts with custom shields settings of normal window

Reproduces how often:

always

Brave version (brave://version info)

every version

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds?
    yes

Website problems only:

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

Additional Information

Currently, this is expected behavior because tor window uses different profile with other normal windows.

@rebron rebron added this to the 1.x Backlog milestone Jan 18, 2019
@rebron rebron added the feature/shields The overall Shields feature in Brave. label Jan 18, 2019
@tildelowengrimm tildelowengrimm added feature/tor feature/tor/guest-semantics Exposing the fact that private windows with Tor are built on guest sessions. priority/P4 Planned work. We expect to get to it "soon". labels Jan 24, 2019
@rebron rebron removed this from the 1.x Backlog milestone Feb 7, 2019
@user809256
Copy link

Is there any fix for this issue?

I kinda don't see point in Tor network when fingerprint and script shields are down by default. I need to enable all shields manually for every website because global settings don't exist for guest/tor user?

Please tell me if there is any workarounds currently for this issue. Thanks ^^

@yrliou
Copy link
Member

yrliou commented Oct 3, 2019

This is fixed by brave/brave-core#3319.

@yrliou yrliou closed this as completed Oct 3, 2019
@yrliou yrliou added this to the 0.72.x - Nightly milestone Oct 3, 2019
@yrliou yrliou added the QA/Yes label Oct 3, 2019
@btlechowski
Copy link

btlechowski commented Oct 28, 2019

Verification passed on

Brave 0.72.110 Chromium: 78.0.3904.70 (Official Build) dev (64-bit)
Revision edb9c9f3de0247fd912a77b7f6cae7447f6d3ad5-refs/branch-heads/3904@{#800}
OS Ubuntu 18.04 LTS

Verified test plan from the description
Verified test plan from brave/brave-core#3319
Verified bookmarks and global settings are shared between normal and tor window. Extensions are not shared yet per #2307 (comment)
Verified that brave://settings page is redirected to normal window.

Verification passed on

Brave 0.72.124 Chromium: 78.0.3904.97 (Official Build) beta (64-bit)
Revision 021b9028c246d820be17a10e5b393ee90f41375e-refs/branch-heads/3904@{#859}
OS Windows 10 OS Version 1803 (Build 17134.1006)

Verified passed with

Brave 1.1.2 Chromium: 78.0.3904.97 (Official Build) beta (64-bit)
Revision 021b9028c246d820be17a10e5b393ee90f41375e-refs/branch-heads/3904@{#859}
OS macOS Version 10.13.6 (Build 17G5019)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature/shields The overall Shields feature in Brave. feature/tor/guest-semantics Exposing the fact that private windows with Tor are built on guest sessions. feature/tor priority/P4 Planned work. We expect to get to it "soon". QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Yes release-notes/include
Projects
None yet
Development

No branches or pull requests

8 participants