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

[Desktop] torrent and magnet link pages show blocked by extension message after browser restart #7330

Closed
LaurenWags opened this issue Dec 11, 2019 · 8 comments · Fixed by brave/brave-core#6085

Comments

@LaurenWags
Copy link
Member

LaurenWags commented Dec 11, 2019

Description

Follow up to #6472

Torrent and magnet link pages show a "blocked by an extension" message after browser restart.

This does not reproduce on C78 versions (1.0.1 Release, 1.2.11 Beta) but does reproduce on C79 versions (1.1.20 Release and 1.2.23 Beta).

Steps to Reproduce

  1. Clean profile.
  2. Open webtorrent.io/free-torrents.
  3. Click on a torrent or magnet (will encounter Blank page for magnet links on clean install #7061 for magnet) link.
  4. Restart Brave.

Actual result:

Get message below. Refresh of page does not help.
Screen Shot 2019-12-11 at 11 40 56 AM
Screen Shot 2019-12-11 at 11 56 33 AM

You can copy/paste the URL into a new tab and that will work.

Expected result:

Torrent page to be displayed.

Reproduces how often:

easily

Brave version (brave://version info)

Brave 1.1.20 Chromium: 79.0.3945.74 (Official Build) (64-bit)
Revision 99b6d241cf84f7fa7434ae8a59f9e74eb1464280-refs/branch-heads/3945@{#888}
OS macOS Version 10.13.6 (Build 17G5019)
Brave 1.2.23 Chromium: 79.0.3945.70 (Official Build) beta (64-bit)
Revision 65868db27916cebde51fc2465c9b34761e6383e2-refs/branch-heads/3945@{#869}
OS macOS Version 10.13.6 (Build 17G5019)

Version/Channel Information:

  • Can you reproduce this issue with the current release? yes reproduces with 1.1.20, did not occur with 1.0.1 though
  • Can you reproduce this issue with the beta channel? yes
  • Can you reproduce this issue with the dev channel? unsure
  • Can you reproduce this issue with the nightly channel? unsure

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:

@rebron
Copy link
Collaborator

rebron commented Feb 7, 2020

This is because we're lazy loading the extension, we need to add a check on startup if there's a tab open.

@rebron rebron added the priority/P4 Planned work. We expect to get to it "soon". label Feb 7, 2020
@feross
Copy link

feross commented Jul 9, 2020

@bbondy Looks like this case was missed when adding lazy-loading extensions. Is this a fix that is needed for other lazy-loaded extensions too? Can we fix it once for all cases?

@kjozwiak
Copy link
Member

kjozwiak commented Jul 9, 2020

+1 from https://twitter.com/LuisNil22/status/1281322799634710528 as per @BrendanEich. A quick note, clicking on the refresh button won't work either. User will need to click in the URL and load it again. Example:

webTorrent

@bbondy
Copy link
Member

bbondy commented Jul 13, 2020

Checking if @darkdh or @yrliou might have some cycles to look at this.

@yrliou yrliou self-assigned this Jul 13, 2020
@yrliou yrliou added this to the 1.13.x - Nightly milestone Jul 16, 2020
@rebron rebron changed the title torrent and magnet link pages show blocked by extension message after browser restart [Desktop] torrent and magnet link pages show blocked by extension message after browser restart Jul 20, 2020
@LaurenWags
Copy link
Member Author

LaurenWags commented Aug 19, 2020

Verified passed with

Brave | 1.13.77 Chromium: 85.0.4183.69 (Official Build) dev (64-bit)
-- | --
Revision | 4554ea1a1171bd8d06951a4b7d9336afe6c59967-refs/branch-heads/4183@{#1426}
OS | macOS Version 10.14.6 (Build 18G3020)
  • Verified STR from description.
  • Confirmed if torrent/magnet link is active tab there is no error shown on the tab on browser restart.
  • Confirmed if torrent/magnet link is non-active tab there is no error shown on the tab on browser restart.
  • Encountered Blank page for magnet links on clean install #7061 while testing

Verification passed on

Brave | 1.13.77 Chromium: 85.0.4183.69 (Official Build) dev (64-bit)
-- | --
Revision | 4554ea1a1171bd8d06951a4b7d9336afe6c59967-refs/branch-heads/4183@{#1426}
OS | Windows 10 OS Version 1903 (Build 18362.1016)

  • Verified STR from description.
  • Confirmed if torrent/magnet link is active tab there is no error shown on the tab on browser restart.
  • Confirmed if torrent/magnet link is non-active tab there is no error shown on the tab on browser restart.
  • Encountered Blank page for magnet links on clean install #7061 while testing

Verification passed on

Brave 1.13.77 Chromium: 85.0.4183.69 (Official Build) dev (64-bit)
Revision 4554ea1a1171bd8d06951a4b7d9336afe6c59967-refs/branch-heads/4183@{#1426}
OS Ubuntu 18.04 LTS
  • Verified STR from description.
  • Confirmed if torrent/magnet link is active tab there is no error shown on the tab on browser restart.
  • Confirmed if torrent/magnet link is non-active tab there is no error shown on the tab on browser restart.
  • Encountered Blank page for magnet links on clean install #7061 while testing

@xlcrr
Copy link

xlcrr commented May 2, 2022

Changing private browsing to normal browsing fixed it for me

@DylanYoung
Copy link

Is there another issue to fix the private browsing case? Either it needs a separate setting or it should just be disabled in private browsing and allow regular download.

@theplaygod97
Copy link

Issue is still present in private browsing tab

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment