Skip to content

Conversation

@papphelix
Copy link

@papphelix papphelix commented Dec 11, 2025

This PR is a fix for streaming m3u8 url video streaming issue which reduces the quality of video and audio after 10sec of play.

Issue: A half dozen learners and partners that many (but not all) videos are slowing down and distorting the audio. Learner Support and Partner Support have been able to reproduce this. It seems to be happening approximately 10 seconds into the impacted videos.

Solution & tested on Stage edx: By upgrading hls.js to 1.6.15 it solves the playback issue.

Impact: All Streaming using m3u8 url.

Screen.Recording.2025-12-08.at.22.09.34-720.mov

Copy link
Contributor

@farhan farhan left a comment

Choose a reason for hiding this comment

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

Thanks for bumping the HLS version and fixing the issue.
I’ve left a query.

@papphelix papphelix requested a review from farhan December 22, 2025 11:02
@papphelix papphelix requested a review from feanil January 23, 2026 12:27
Copy link
Contributor

@feanil feanil left a comment

Choose a reason for hiding this comment

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

@papphelix has this change been merged into edx-platform master already? For fixes to release branches we required that they're reviewed and merged into master before we backport them to releases. Can you link to the edx-platform PR here so we can make sure that it merges before this merges.

@papphelix papphelix changed the base branch from release/ulmo to master January 24, 2026 11:27
@papphelix papphelix changed the base branch from master to release/ulmo January 24, 2026 11:27
@papphelix
Copy link
Author

@papphelix has this change been merged into edx-platform master already? For fixes to release branches we required that they're reviewed and merged into master before we backport them to releases. Can you link to the edx-platform PR here so we can make sure that it merges before this merges.

Raised a new PR #37936
closing this as suggested

@papphelix papphelix closed this Jan 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants