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

Move Tor settings to Privacy and security section (brave://settings/privacy) #16624

Closed
rebron opened this issue Jun 25, 2021 · 4 comments
Closed

Comments

@rebron
Copy link
Collaborator

rebron commented Jun 25, 2021

Description

The extensions section of brave://settings is overused and cluttered.

Move Tor settings into Privacy and security section.
brave://settings/privacy

New and Expected Design

image-2

Existing Design

Screen Shot 2021-06-25 at 12 54 32 PM

Brave version (brave://version info)

Version/Channel Information:

  • Can you reproduce this issue with the current release? n/a
  • Can you reproduce this issue with the beta channel? n/a
  • Can you reproduce this issue with the nightly channel? n/a

Other Additional Information:

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

Miscellaneous Information:

Figma: https://www.figma.com/file/tLXWGCpNoiJxDZDdpfordj/Desktop-Settings?node-id=2187%3A60437

@aguscruiz
Copy link

aguscruiz commented Jun 1, 2022

Updated design here:
https://www.figma.com/file/tLXWGCpNoiJxDZDdpfordj/Desktop-Settings?node-id=2187%3A60437

I do think this looks pretty bare bones, so I wonder if we couldn't group these settings with another section

@rebron rebron changed the title Add section in brave://settings for Tor options move Tor settings to Privacy and security section brave://settings/privacy Jul 12, 2022
@rebron rebron changed the title move Tor settings to Privacy and security section brave://settings/privacy move Tor settings to Privacy and security section (brave://settings/privacy) Jul 12, 2022
@rebron rebron changed the title move Tor settings to Privacy and security section (brave://settings/privacy) move Tor settings to Privacy and security section (brave://settings/privacy) Jul 12, 2022
@rebron
Copy link
Collaborator Author

rebron commented Aug 9, 2022

Fixed with brave/brave-core#13290

@rebron rebron closed this as completed Aug 9, 2022
@rebron rebron added this to the 1.44.x - Nightly milestone Aug 9, 2022
@stephendonner
Copy link

stephendonner commented Sep 19, 2022

Verification PASSED using

Brave 1.44.92 Chromium: 105.0.5195.136 (Official Build) beta (64-bit)
Revision 872774b783d0e674186a3adcd2f92e7aa22a219c-refs/branch-heads/5195_124@{#4}
OS Windows 10 Version 21H2 (Build 19044.2006)

Settings/UI

Case 1: New Tor windows sub-section of Privacy & Security - PASSED

Confirmed a new Tor windows section exists in brave://settings/privacy, with the following preferences and default values:

  • Private window with Tor - ON
  • Automatically redirect .onion sites - OFF
  • Use Bridges - OFF

tor-settings-ui

Case 2: Toggle Private window with Tor to OFF - PASSED

Confirmed when Private window with Tor is disabled, Automatically redirect .onion sites and Use Bridges are disabled

private-window-to-toggled-off

Case 3: Enabling Use Bridges should expand to additional settings - PASSED

Confirmed toggling Use Bridges to Enabled expands to the following three radio-button options:

  • Select a built-in bridge with obfs4 selected
  • Request a bridge from torproject.org w/read-only textfield and Request a New Bridge... button
  • Provide a bridge with input textfield
  • also confirmed toggling it to Disabled collapses it to its original state

use-bridges-open

Case 4: Apply changes button should be dynamic - PASSED

Confirmed the following:

  • the Apply changes button appears after changing input values and/or radio-button selections (with input)
  • after clicking Apply changes, the button disappears until another change is made
  • the Apply changes button correctly disappears if a value was changed (to trigger the button) but not /applied/, and then reverted to its original value
Select a built-in-bridge Request a bridge from torproject.org Provide a bridge
apply-changes-1 apply-changes-2 apply-changes-3

Bridges (basic)

Case 1: Built-in bridges listing - PASSED

Confirmed built-in bridges are the following, in order:

  • obfs4
  • Snowflake
  • meek-azure

bridges-listing

Case 2: Built-in bridges functionality - PASSED

Confirmed the steps below with each of the following built-in bridges:

  • obfs4
  • Snowflake
  • meek-azure
  1. selected a bridge
  2. clicked on Apply changes
  3. clicked on "hamburger menu" -> New private window with Tor
  4. confirmed I could load check.torproject.org
  5. double-confirmed by clicking on New Tor connection for this site, and ensuring it reloads
obfs4 Snowflake meek-azure
obfs4 Snowflake meek-azure

Case 3: Request a bridge from torproject.org - PASSED

Confirmed the following:

  • empty, read-only textfield
  • placeholder text on selection reads Click on "Request a New Bridge..." to get bridges
  • clicking on the Request a bridge from torproject.org radio button enables the Request a New Bridge... button
  • clicking the Request a New Bridge button displays a Request Bridge CAPTCHA dialog
Request a bridge from... radio button Request a New Bridge... button
request-1 request-2

Case 3.5 Request Bridge CAPTCHA dialog - PASSED

Confirmed the following:

  • X dismisses the dialog
  • Cancel dismisses the dialog
  • clicking the Renew button refreshes the CAPTCHA
  • invalid input refreshes the CAPTCHA
  • CAPTCHA solves are case-insensitive
  • valid CAPTCHA submission dismisses the dialog and populates the read-only textarea with valid obfs4 bridges (obfs4 xxx.xxx.xxx:xxxx... lines)
Renew button invalid input valid obfs4 bridges
1 2 request-3

Logged:

Case 4: Provide a bridge - PASSED

Confirmed the following:

  • empty input textarea
  • placeholder text on selection reads Type address:port (one per line)
  • used previous config from Request a Bridge... test
  • also used configs generated from https://bridges.torproject.org/options/
focused/selected check.torproject.org
provide1 request-5

Bridges (Detailed)

Case 1: Brave Tor Client Updater component - PASSED

  • opened brave://components
  • launched New private window with Tor
  • confirmed Brave Tor Client Updater appears
  • confirmed new 1.0.54 version (previous 1.0.28)

component

Case 2: Brave Pluggable Transports component - PASSED

  • opened brave://settings/privacy
  • clicked to toggle Use Bridges to Enabled
  • clicked Apply changes
  • opened brave://components
  • confirmed new Brave Pluggable Transports (Windows) appears
  • noted version 1.0.3

pluggable-transports

Case 3: Specific transport-binary processes - PASSED

obfs4

  • continued from the above testcase
  • in brave://settings/privacy, chose obfs4 (and then again with meek-azure chosen)
  • clicked Apply changes
  • opened a New private window with Tor
  • loaded check.torproject.org
  • opened the OS Task Manager, and confirmed the process name
brave://settings/privacy check.torproject.org tor-obfs4-brave
obfs4 obfs4 -2png tor-obfs4-brave

Snowflake

  • in brave://settings/privacy, chose Snowflake
  • clicked Apply changes
  • opened a New private window with Tor
  • loaded check.torproject.org
  • opened the OS Task Manager, and confirmed the process name
brave://settings/privacy check.torproject.org tor-snowflake-brave
snowflake-1 snowflake-2 tor-snowflake-brave

meek-azure

  • in brave://settings/privacy, chose meek-azure
  • clicked Apply changes
  • opened a New private window with Tor
  • opened the OS Task Manager, and confirmed the process name
brave://settings/privacy check.torproject.org tor-obfs4-brave
meek-azure-1 meek-azure-2 meek-azure-tor-obfs4-brave

Case 4: Disabling bridges should stop processes and remove components from profile - PASSED

  • ran through the above cases for each bridge type again, as setup
  • toggled Use Bridges to Disabled
  • clicked Apply changes
  • examined: Task Manager and {user dir} for

Confirmed:

  • no bridge-specific processes still running
  • bridge-transport component directory dnkcahhmfcanmkjhnjejoomdihffoefm is deleted

Running bridges

obfs4 Snowflake meek-azure
obfs4-running snowflake-running meek-azure-running

Disabled bridges

obfs4 Snowflake meek-azure
obfs4-shutdown snowflake-shutdown meek-azure-shutdown

Case 5: Changing bridge settings should renew IP address - PASSED

  • toggled Use Bridges to Enabled on brave://settings/privacy
  • clicked Apply changes
  • clicked on "hamburger menu" -> New private window with Tor
  • loaded yandex.com/internet
  • noted the IP address - 185.14.97.176
  • selected a new bridge type on brave://settings/privacy
  • clicked Apply changes
  • reload the yandex.com/internet tab
  • noted the IP address - 185.220.101.180

Confirmed I was assigned a new IP address 185.220.101.180, replacing 185.14.97.176 after changing bridge types from obfs4 to Snowflake

obfs4 bridge obfs4 IP Snowflake bridge Snowflake IP
changing-1 changing-2 changing-3 changing-4

Case 6: Bad bridge config - PASSED

  • in brave://settings/privacy, pasted obfs4 99.124.226.90:9998 8BFDDB7D7D2D4BDD4169170C818B175C6B60F799 cert=fmp+hf7s1QH6Crg+FW39P5KTxy57NCfXs+t1vBrNuYrHGebGtbRrdkKfk+pcgZhBdbrVcw iat-mode=0 (from the provided testplan) into the Provide a bridge textarea
  • clicked Apply changes
  • clicked "hamburger menu" -> New private window with Tor
  • loaded brave.com
  • waited

Confirmed I got the Tor connection failed error message

  • clicked on the Brave Settings and bridges links, which go to brave://settings/privacy
  • clicked on the contact support link and confirmed it loaded https://support.brave.com/hc/en-us in a new tab
brave://settings/privacy Tor connection failed support.brave.com
bad-bridge-0 bad-bridge-1 support

@stephendonner stephendonner added QA/In-Progress Indicates that QA is currently in progress for that particular issue QA Pass-Win64 and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Sep 19, 2022
@stephendonner stephendonner added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Sep 26, 2022
@stephendonner
Copy link

stephendonner commented Sep 26, 2022

Verification PASSED using

Brave 1.44.99 Chromium: 106.0.5249.55 (Official Build) (x86_64)
Revision 4d5f098fca6ab7f4b6b7c240be3d9593c2357709-refs/branch-heads/5249@{#531}
OS macOS Version 11.7 (Build 20G817)

Settings UI

Case 1: New Tor windows sub-section of Privacy & Security - PASSED

* Private window with Tor - `On` * Automatically redirect .onion sites` `Off` * Use Bridges `Off` Screen Shot 2022-09-26 at 11 30 57 AM

Case 2: Toggling Private window with Tor to Off - PASSED

Confirmed when Private window with Tor is disabled, Automatically redirect .onion sites and Use Bridges are disabled

Screen Shot 2022-09-26 at 12 00 55 PM

Case 3: Enabling Use Bridges should expand to additional settings - PASSED

Confirmed toggling Use Bridges to Enabled expands to the following three radio-button options:

  • Select a built-in bridge with obfs4 selected
  • Request a bridge from torproject.org w/read-only textfield and Request a New Bridge... button
  • Provide a bridge with input textfield
    also confirmed toggling it to Disabled collapses it to its original state
Screen Shot 2022-09-26 at 12 03 21 PM

Case 4: Apply changes button should be dynamic - PASSED

Confirmed the following:

  • the Apply changes button appears after changing input values and/or radio-button selections (with input)
  • after clicking Apply changes, the button disappears until another change is made
  • the Apply changes button correctly disappears if a value was changed (to trigger the button) but not /applied/, and then reverted to its original value
Select a built-in-bridge Request a bridge from torproject.org Provide a bridge
Screen Shot 2022-09-26 at 12 14 05 PM Screen Shot 2022-09-26 at 12 14 56 PM Screen Shot 2022-09-26 at 12 15 07 PM

Bridges (basic)

Case 1: Built-in bridges listing - PASSED

Confirmed built-in bridges are the following, in order:

  • obfs4
  • Snowflake
  • meek-azure
Screen Shot 2022-09-26 at 12 21 07 PM
Case 2: Built-in bridges functionality - `PASSED`
#### Confirmed the steps below with each of the following built-in bridges:
  • obfs4
  • Snowflake
  • meek-azure
  1. selected a bridge
  2. clicked on Apply changes
  3. clicked on "hamburger menu" -> New private window with Tor
  4. confirmed I could load check.torproject.org
  5. double-confirmed by clicking on New Tor connection for this site, and ensuring it reloads
obfs4 Snowflake meek-azure
Screen Shot 2022-09-26 at 12 36 14 PM Screen Shot 2022-09-26 at 12 39 36 PM Screen Shot 2022-09-26 at 12 43 57 PM
Screen Shot 2022-09-26 at 12 43 57 PM

Case 3: Request a bridge from torproject.org - PASSED

#### Confirmed the following:
  • empty, read-only textfield
  • placeholder text on selection reads Click on "Request a New Bridge..." to get bridges
  • clicking on the Request a bridge from torproject.org radio button enables the Request a New Bridge... button
  • clicking the Request a New Bridge button displays a Request Bridge CAPTCHA dialog
Request a bridge from... radio button Request a New Bridge... button
Screen Shot 2022-09-26 at 12 51 35 PM Screen Shot 2022-09-26 at 12 51 40 PM

Case 3.5: Request Bridge CAPTCHA dialog - PASSED

Confirmed the following:

  • X dismisses the dialog
  • Cancel dismisses the dialog
  • clicking the Renew button refreshes the CAPTCHA
  • invalid input refreshes the CAPTCHA
  • CAPTCHA solves are case-insensitive
  • valid CAPTCHA submission dismisses the dialog and populates the read-only textarea with valid obfs4 bridges (obfs4 xxx.xxx.xxx:xxxx... lines)
Renew button invalid input valid obfs4 bridges
Screen Shot 2022-09-26 at 1 03 30 PM Screen Shot 2022-09-26 at 1 07 24 PM Screen Shot 2022-09-26 at 1 08 06 PM

Case 4: Provide a bridge - PASSED

#### Confirmed the following:
  • empty input textarea
  • placeholder text on selection reads Type address:port (one per line)
  • used previous config from Request a Bridge... test
  • also used configs generated from https://bridges.torproject.org/options/
focused/selected check.torproject.org
Screen Shot 2022-09-26 at 1 13 31 PM Screen Shot 2022-09-26 at 1 14 23 PM

Bridges (detailed)

Case 1: Brave Tor Client Updater component - PASSED

  • opened brave://components
  • launched New private window with Tor
  • confirmed Brave Tor Client Updater appears
  • confirmed new 1.0.28 version (previous ``)
Screen Shot 2022-09-26 at 1 31 30 PM

Case 2: Brave Pluggable Transports component - PASSED

opened brave://settings/privacy
clicked to toggle Use Bridges to Enabled
clicked Apply changes
opened brave://components
confirmed new Brave Pluggable Transports (Mac) appears
noted version 1.0.0

Screen Shot 2022-09-26 at 1 33 26 PM

Case 3: Specific transport-binary processes - PASSED

obfs4

  • continued from the above testcase
  • in brave://settings/privacy, chose obfs4
  • clicked Apply changes
  • opened a New private window with Tor
  • loaded check.torproject.org
  • opened the OS Task Manager, and confirmed the process name
brave://settings/privacy check.torproject.org tor-obfs4-brave
Screen Shot 2022-09-26 at 1 41 10 PM Screen Shot 2022-09-26 at 1 42 14 PM Screen Shot 2022-09-26 at 1 48 00 PM

Snowflake

  • in brave://settings/privacy, chose Snowflake
  • clicked Apply changes
  • opened a New private window with Tor
  • loaded check.torproject.org
  • opened the OS Task Manager, and confirmed the process name
brave://settings/privacy check.torproject.org tor-snowflake-brave
Screen Shot 2022-09-26 at 1 49 04 PM Screen Shot 2022-09-26 at 2 00 36 PM Screen Shot 2022-09-26 at 1 58 56 PM

meek-azure

  • in brave://settings/privacy, chose meek-azure
  • clicked Apply changes
  • opened a New private window with Tor
  • opened the OS Task Manager, and confirmed the process name
brave://settings/privacy check.torproject.org tor-obfs4-brave
Screen Shot 2022-09-26 at 2 05 31 PM Screen Shot 2022-09-26 at 2 09 52 PM Screen Shot 2022-09-26 at 2 07 43 PM

Logged:

Case 4: Disabling bridges should stop processes and remove components from profile - PASSED

  • ran through the above cases for each bridge type again, as setup
  • toggled Use Bridges to Disabled
  • clicked Apply changes
  • examined: Activity Monitor and {user dir} for

Confirmed:

  • no bridge-specific processes still running
  • bridge-transport component directory

Running bridges

obfs4 Snowflake meek-azure
Screen Shot 2022-09-26 at 3 23 07 PM Screen Shot 2022-09-26 at 3 13 15 PM Screen Shot 2022-09-26 at 3 17 38 PM

Disabled bridges

obfs4 Snowflake meek-azure
Screen Shot 2022-09-26 at 3 50 32 PM Screen Shot 2022-09-26 at 3 53 29 PM Screen Shot 2022-09-26 at 3 56 56 PM

Case 5: Changing bridge settings should renew IP address - PASSED

  • toggled Use Bridges to Enabled on brave://settings/privacy
  • clicked Apply changes
  • clicked on "hamburger menu" -> New private window with Tor
  • loaded yandex.com/internet
  • noted the IP address - 185.195.71.3
  • selected a new bridge type on brave://settings/privacy
  • clicked Apply changes
  • reload the yandex.com/internet tab
  • noted the IP address

Confirmed I was assigned a new IP address 184.75.221.59

, replacing 185.195.71.3 after changing bridge types from obfs4 to Snowflake

obfs4 bridge obfs4 IP Snowflake bridge Snowflake IP
Screen Shot 2022-09-26 at 2 39 02 PM Screen Shot 2022-09-26 at 2 40 26 PM Screen Shot 2022-09-26 at 2 39 02 PM Screen Shot 2022-09-26 at 2 34 56 PM

Case 6: Bad bridge config - PASSED

  • in brave://settings/privacy, pasted obfs4 99.124.226.90:9998 8BFDDB7D7D2D4BDD4169170C818B175C6B60F799 cert=fmp+hf7s1QH6Crg+FW39P5KTxy57NCfXs+t1vBrNuYrHGebGtbRrdkKfk+pcgZhBdbrVcw iat-mode=0 (from the provided https://github.com/brave/brave-core/pull/13290#issue-1232508125) into the Provide a bridge textarea
  • clicked Apply changes
  • clicked "hamburger menu" -> New private window with Tor
  • loaded brave.com
  • waited

Confirmed I got the Tor connection failed error message

  • clicked on the Brave Settings and bridges links, which go to brave://settings/privacy
  • clicked on the contact support link and confirmed it loaded https://support.brave.com/hc/en-us in a new tab
brave://settings/privacy Tor connection failed support.brave.com
Screen Shot 2022-09-26 at 4 08 04 PM Screen Shot 2022-09-26 at 4 09 06 PM Screen Shot 2022-09-26 at 4 11 10 PM

@rebron rebron changed the title move Tor settings to Privacy and security section (brave://settings/privacy) Move Tor settings to Privacy and security section (brave://settings/privacy) Sep 26, 2022
@LaurenWags LaurenWags removed the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Sep 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants