Skip to content

[Lily/Video] add m3u8 support #1785

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.

Sign up for GitHub

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

Draft
wants to merge 12 commits into
base: master
Choose a base branch
from
+86 −3
Draft

[Lily/Video] add m3u8 support #1785

Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
da35819
Lily/Video add m3u8 support
jexjws Dec 11, 2024
48c5e3c
Lily/Video auto load hls.js from cdn.jsdelivr.net
jexjws Dec 11, 2024
47536e5
Merge branch 'TurboWarp:master' into master
jexjws Dec 11, 2024
3cf1071
remove chinese comment
jexjws Dec 11, 2024
d0cf9a2
Update Video.js
jexjws Dec 12, 2024
3d8cdc7
npm run format
jexjws Dec 12, 2024
0f64b1a
Lily/Video bundle hls.js directly into the extension
jexjws Dec 12, 2024
0b9215a
Lily/Video comment out console.log
jexjws Dec 12, 2024
21bb391
hotfix
jexjws Dec 12, 2024
b845e70
fix npm run lint
jexjws Dec 13, 2024
487b577
Add a guide for reviewing HJSLibCompressed.
jexjws Dec 13, 2024
1134ad7
Merge branch 'TurboWarp:master' into master
jexjws Dec 22, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
89 changes: 86 additions & 3 deletions extensions/Lily/Video.js
View file
Open in desktop
Copy link
Author

@jexjws jexjws Dec 11, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more. 

if (!(await Scratch.canFetch(url))) return;

WARNING : If the resource URLs in a .m3u8 file are not on the same domain as the .m3u8 file itself, the client can fetch resources from other domains without using Scratch.fetch.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps you can do a check using a custom loader for hls.js before a playlist or fragment gets loaded. This is a quick mock up but haven't tested it at all:

this.hls = new Hls({
  loader: class CustomLoader extends Hls.DefaultConfig.loader {
    load(context, config, callbacks) {
      let { type, url } = context;

      // Custom behavior - do a check first
      Scratch.canFetch(url).then(isAllowed => {
        if (!isAllowed) {
          callbacks.onErrorCallback({
            code: 'Scratch_Error',
            text: `${err}`
          });
        } else {
          super.load(context, config, callbacks);
        }
      }).catch(err => callbacks.onErrorCallback({
        code: 'Scratch_Error',
        text: `${err}`
      }));
    }
  }
});

Large diffs are not rendered by default.

Loading