-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
WebTorrent: Video streaming is broken since upgrade to WebTorrent v0.105.0 #5358
WebTorrent: Video streaming is broken since upgrade to WebTorrent v0.105.0 #5358
Comments
There's a new release of |
Re-opening until brave/brave-core#2989 is merged. |
Moving this into the |
This is fixed in 0.69.x too now because we upgrade WebTorrent to 0.105.1 in 0.69.x. |
@yrliou Thank you for debugging this issue! |
Verification passed on
Verified test plan from the description. Was able to stream the video Verified passed with
Verification passed on
|
Description
It appears that WebTorrent video playback is broken on Nightly, ever since the upgrade to WebTorrent v0.105.0.
The
URL
constructor is undefined on this line inwebtorrent/lib/server.js
:https://github.com/webtorrent/webtorrent/blob/78d98d9fa45598b63fb0fec013036e35d167f6c3/lib/server.js#L80
The reason appears that webpack replaces this line:
https://github.com/webtorrent/webtorrent/blob/78d98d9fa45598b63fb0fec013036e35d167f6c3/lib/server.js#L6
with:
and the
url
package doesn't exportURL
asurl.URL
even though it exists in the chrome extension environment. I can fix this on thewebtorrent
side and publish a new version.cc @yrliou
Steps to Reproduce
brave://inspect/#extensions
) to see the exception.Actual result:
Expected result:
Video plays
Reproduces how often:
Easily reproduced
Brave version (brave://version info)
Nightly. Version 0.69.77 Chromium: 76.0.3809.62 (Official Build) nightly (64-bit)
Version/Channel Information:
The text was updated successfully, but these errors were encountered: