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

Creating a new Tor window using the purple Tor pill button in the URL bar fails to show tor daemon status #37927

Closed
urbenlegend opened this issue Apr 29, 2024 · 7 comments · Fixed by brave/brave-core#23410

Comments

@urbenlegend
Copy link

Description

When clicking on the purple Tor button in the URL bar to navigate to the Tor version of a specific site, the page never seems to start loading. It's like the Tor session never connects. No progress spinner is displayed on the Tor tab. Only after opening a new blank tab in the Tor window and then going back to the previous tab and refreshing will the Tor page load.

NOTE: This does not happen if there is already an open Tor window. This bug only triggers when no previous Tor session exists.

Steps to Reproduce

  1. Make sure all Tor windows are closed
  2. Go to nytimes.com
  3. Click the Tor button in the URL bar
  4. Notice that a new Tor window pops up, but nothing loads. The title of the page remains "New Private Tab".
  5. Open a new blank tab in the same Tor window. Make sure it says Tor is connected.
  6. Go back to previous tab and refresh. Page will now load.

Actual result:

If a user clicks on the purple Tor button and no previous Tor session exists, Tor page refuses to load

Expected result:

Purple Tor button should reliably open the tor page, even if it has to create a new Tor session to do so.

Reproduces how often:

Every time

Brave version (brave://version info)

1.65.123 Chromium: 124.0.6367.91 (Official Build) (64-bit)

Version/Channel Information:

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

Other Additional Information:

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

Miscellaneous Information:

@rebron rebron assigned rebron and boocmp and unassigned rebron Apr 30, 2024
@rebron
Copy link
Collaborator

rebron commented Apr 30, 2024

cc: @fmarier @MadhaviSeelam

@fmarier
Copy link
Member

fmarier commented Apr 30, 2024

For me, it does eventually load, but there is no feedback at all during the bootstrapping phase (which can take a long time).

Tested with brave.com and then clicking on the purple Tor button in the URL bar

@MadhaviSeelam
Copy link

Verified with both brave.com & nytimes.com on 1.65.126 on Win 11. Noticed as below:

  • Tor window is blank after clicking Tor button
  • brave.com loaded consistently between 40 sec - 60 sec. (tested 2x)
  • nytimes.com either loads successfully or fails after ~60 secs. (tested ~4x)

brave.com

brave.mp4

nytimes.com

2024-05-02_10h08_40.mp4

@urbenlegend
Copy link
Author

As of Brave version 1.65.126 Chromium: 124.0.6367.118 (Official Build) (64-bit), it does seem to eventually load. Maybe the failed load I experienced before and what MadhaviSeelam experienced was just typical Tor slowness?

I did a timing test for nytimes.com and they seem to line up okay:

  1. Clicking purple Tor button without a pre-existing Tor window: 53 sec
  2. Opening Tor window, waiting for connect, and then clicking purple Tor button: 50 sec

But yeah, I think there does need to be some loading feedback. Maybe the Tor private browsing intro page should be shown first and then once it connects transition over to loading the destination page?

@fmarier fmarier changed the title Creating a new Tor window using the purple Tor pill button in the URL bar fails to load page Creating a new Tor window using the purple Tor pill button in the URL bar fails to show tor daemon status May 2, 2024
@fmarier
Copy link
Member

fmarier commented May 2, 2024

I think there does need to be some loading feedback.

Agreed. I've retitled the issue to that effect.

@fmarier fmarier added the priority/P4 Planned work. We expect to get to it "soon". label May 2, 2024
@boocmp
Copy link

boocmp commented May 3, 2024

I posted a video in the PR, could you please take a look? Does it work as you expect, or does it need to be adjusted?

@stephendonner
Copy link

stephendonner commented Jun 11, 2024

Verification PASSED using

Brave | 1.68.85 Chromium: 126.0.6478.40 (Official Build) beta (64-bit)
-- | --
Revision | 8e76a669cf78bdfa933c8d569460092a01166588
OS | Windows 10 Version 22H2 (Build 19045.4474)

Steps:

  1. installed 1.68.85
  2. launched Brave
  3. loaded `nytimes.com
  4. clicked on the purple Tor button in the URL bar
  5. confirmed I saw loading status (page URL, page title) in the upper-left tab
example example example example example
image image image image image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Completed
Development

Successfully merging a pull request may close this issue.

7 participants