Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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.

# for free to join this conversation on GitHub. Already have an account? # 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.