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

Provide a switch to enable / disable WebTorrent in settings #897

Closed
yrliou opened this issue Aug 31, 2018 · 49 comments · Fixed by brave/brave-core#1218
Closed

Provide a switch to enable / disable WebTorrent in settings #897

yrliou opened this issue Aug 31, 2018 · 49 comments · Fixed by brave/brave-core#1218
Assignees
Labels
design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. design A design change, especially one which needs input from the design team feature/webtorrent Label for webtorrent related issues priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Yes release-notes/include

Comments

@yrliou
Copy link
Member

yrliou commented Aug 31, 2018

We should provide a switch for webtorrent, probably in a brave-specific settings section.

@yrliou yrliou added the feature/webtorrent Label for webtorrent related issues label Aug 31, 2018
@bbondy bbondy added this to the Backlog milestone Aug 31, 2018
@rebron rebron added design A design change, especially one which needs input from the design team design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. labels Oct 22, 2018
@rebron rebron self-assigned this Oct 22, 2018
@yrliou yrliou self-assigned this Nov 3, 2018
@eljuno
Copy link

eljuno commented Nov 4, 2018

@NejcZdovc
Copy link
Contributor

+1 from me

@lordirfan99
Copy link

+1 from me. i would like to use 3rd party torrent apps

@pongraczi
Copy link

+1
In general, it should be a good idea to not force the user to use an extension like this, but make it as an option and user could decide.
For me, I even cannot download the torrent file from a site, I have to use chromium or firefox only for download a torrent file.
Thnx

@DarkDibblez
Copy link

+1 from me

@justmwa
Copy link

justmwa commented Nov 24, 2018

+1. Opt-in by default for an optional feature is bad design. No opt-out ability is wtf.

@AoDev
Copy link

AoDev commented Nov 24, 2018

Same complain. I've told people to start using Brave then this came up. Nobody using the browser until this is fixed.

@damienOlsen
Copy link

This is really impacting my transition from Chrome to Brave.
Can we please get a flag to disable this, or just revert to default protocol handling

@Ronkn
Copy link

Ronkn commented Nov 26, 2018

+1 would love to be able to toggle this off or have a work around in the meantime until the fix is pushed to stable. Thanks

@Brave-Matt
Copy link

+1 from Reddit (several users requesting this in the thread):
https://www.reddit.com/r/brave_browser/comments/a0dl7y/torrents_with_brave/

@psypher246
Copy link

Due to lack of switch to turn of webtorrent and this bug: https://community.brave.com/t/webtorrent-in-incognito-mode/37461 NO torrent files can be downloaded in incognito mode. It was mentioned here https://community.brave.com/t/possibility-of-disabling-webtorrent/35699 on Oct 23rd that a switch to turn off webtorrents is coming. What is the progress of that?

@Brave-Matt
Copy link

+1 from Reddit:
https://www.reddit.com/r/brave_browser/comments/a0u7ro/how_do_i_open_torrents_with_my_default_torrent/

@rebron rebron added the priority/P3 The next thing for us to work on. It'll ride the trains. label Nov 27, 2018
@rht
Copy link

rht commented Nov 29, 2018

I tried launching brave by adding a command line flag --disable-webtorrent-extension (which should have worked according to https://github.com/brave/brave-core/blob/46601383bbc6da47c0ac60f0ba0bc5613de2944e/common/brave_switches.cc#L30) but it had no effect on 0.56.12 on NixOS (from a .deb file). It might have worked on other platforms / versions.

Update: ok, the switch was added after 0.56.12, brave/brave-core@25bb585#diff-a9d2a0787e829b9ff4f75ffff0682354
Update2: the switch is confirmed to work in 0.58.10 dev.

@InVItroVeritas
Copy link

+1 from me

@peyao
Copy link

peyao commented Dec 2, 2018

+1, prefer to use my own client with SOCKS5 proxy.

Right now the experience is copy and pasting magnet links, which gets old pretty quickly.

@srirambv
Copy link
Contributor

srirambv commented Dec 3, 2018

+1 from https://community.brave.com/t/trying-to-use-remote-torrent-adder-extension/38775/

cc: @rebron need to prioritize this. Lots of community feedback to provide a switch.

@Brave-Matt
Copy link

Several more +1's from Reddit:
reddit.com/r/brave_browser/comments/9rmnlv/how_can_i_disable_the_built_in_torrent_client/

@Brave-Matt
Copy link

@pongraczi
Copy link

What if developers just disable the torrent handler, until the on/off button will be ready?
I do not know the details, maybe disabling it (for example handling mime/torrrrentttt instead of mime/torrent if complete disable more difficult) is not as difficult as making a new menu/design/button, whatever and could be a quick patch.

@rht
Copy link

rht commented Dec 7, 2018

Since the command line switch already existed, adding it to brave://flags, should be straightforward:

diff --git a/chrome/browser/about_flags.cc b/chrome/browser/about_flags.cc
index 9b1ce28ebb..1ff69333b0 100644
--- a/chrome/browser/about_flags.cc
+++ b/chrome/browser/about_flags.cc
@@ -1284,6 +1284,10 @@ const FeatureEntry::FeatureVariation kOfflineIndicatorFeatureVariations[] = {
 //
 // When adding a new choice, add it to the end of the list.
 const FeatureEntry kFeatureEntries[] = {
+    {"disable-webtorrent-extension",
+     "Disable webtorrent extension",
+     "Disable webtorrent extension.",
+     kOsAll, SINGLE_VALUE_TYPE("disable-webtorrent-extension")}
     {"ignore-gpu-blacklist", flag_descriptions::kIgnoreGpuBlacklistName,
      flag_descriptions::kIgnoreGpuBlacklistDescription, kOsAll,
      SINGLE_VALUE_TYPE(switches::kIgnoreGpuBlacklist)},

At least the about flag should be added until the switch is moved to the web preferences.

@SSCBryce
Copy link

cant believe this is a thing. considering this is an issue from 2018, pretty sure im gonna drop brave now. someone go ahead and message me if this changes

@kjozwiak
Copy link
Member

cant believe this is a thing. considering this is an issue from 2018, pretty sure im gonna drop brave now. someone go ahead and message me if this changes

@Farfie what's the exact issue that you're having? The ability to turn off WebTorrent has been available under brave://settings/extensions for a long time. It comes enabled by default but you can always disable it via the settings. I double checked and made sure it was working as expected.

torrent

You can see that I have it disabled on my profile. Either way, please let me know what the issues you're having.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. design A design change, especially one which needs input from the design team feature/webtorrent Label for webtorrent related issues priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Yes release-notes/include
Projects
None yet
Development

Successfully merging a pull request may close this issue.