-
Notifications
You must be signed in to change notification settings - Fork 970
Fixed download of torrent files #8452
Comments
Updated to be This does need to be fixed though- but I don't believe it should block the release |
@bsclifton why I think it should be blocking is if you don't want to use torrent viewer because of X reason, you can't use torrent files at all. |
@NejcZdovc Good point- are you able to right click and choose "Save Link As..."? I believe this should work The fix for this issue may be easy... WebTorrent logic (filtering, etc) should be ignored if the extension is disabled. For example, the code in https://github.com/brave/browser-laptop/blob/master/app/browser/webtorrent.js |
@bsclifton "Save Link As..." is not working, because we get the same error as we did before I implemented this one #8246 for a regular download in torrent viewer. So we need to fix this scenario as well |
Created a new issue for it #8463 |
Resolves brave#8452 Auditors: @bsclifton Test Plan: - disable Torrent Viewer in preferences - go to https://webtorrent.io/free-torrents/ - download Big Buck Bunny torrent file
Fixes torrent download when torrent viewer is disabled
Resolves #8452 Auditors: @bsclifton Test Plan: - disable Torrent Viewer in preferences - go to https://webtorrent.io/free-torrents/ - download Big Buck Bunny torrent file
Did you search for similar issues before submitting this one?
yes
Describe the issue you encountered:
When you disable Torrent Viewer, torrent download is still routed through it
Platform (Win7, 8, 10? macOS? Linux distro?):
macOS
Brave Version (revision SHA):
b2e4b71
Steps to reproduce:
Actual result:

Expected result:
torrent is downloaded
Will the steps above reproduce in a fresh profile? If not what other info can be added?
yes
Is this an issue in the currently released version?
no
Can this issue be consistently reproduced?
yes
Any related issues:
Support .torrent files in Torrent Viewer #7351
The text was updated successfully, but these errors were encountered: