-
-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[BUG]: 608/708 captions in HLS source not recognized by the player #3766
Comments
@TylerBKrumme any sample video you can share ? |
@freeboub Here's a sample source: https://api.v3.invintus.com/StreamURI/hls/1000000000/2024051020/media.m3u8 |
Thank you for the sample ! Here is the logcat output of the sample which displaying all tracks. I don't see any CEA 608 / 708 tracks..
|
I will close the issue as it is either a stream issue or an exoplayer issue. (cannot be fixed in this repo) May need to be tripple check and investigated with exoplayer sample |
@freeboub By editing react-native-video/android/src/main/java/com/brentvatne/exoplayer/ReactExoplayerView.java in node_modules, we were able to register & display our 608/708 captions.
A prop or build opt that can be used to set .setAllowChunklessPreparation would fix this issue. |
I propose to add a new prop: |
I just created a pull request to integrate the feature: #3882. |
@freeboub Hey, Just got a chance to test it and it doesn't appear to work. Setting source like:
|
Reopen, let me double check |
@TylerBKrumme Finnally review the behavior, Can you test this PR please ? #3913 |
Fix merged for 6.3.0 |
Version
^6.0.0-beta.8
What platforms are you having the problem on?
Android
Architecture
Old architecture
What happened?
608/708 captions within an HLS source aren't recognized by the player, onTextTracks never triggers.
Reproduction
repository link
Reproduction
Ran on an android emulator on a windows PC.
Step to reproduce this bug are:
Set up a basic app with the player, with HLS sources enabled.
Set source to an HLS playlist with embedded 608/708 captions
608/708 tracks are never found, onTextTracks never triggers
The text was updated successfully, but these errors were encountered: