Skip to content

fix: Add MP4 fallback when HLS playback fails - #2

Open
wbaxterh wants to merge 4 commits into
mainfrom
fix/video-player-hls-fallback
Open

fix: Add MP4 fallback when HLS playback fails#2
wbaxterh wants to merge 4 commits into
mainfrom
fix/video-player-hls-fallback

Conversation

@wbaxterh

@wbaxterh wbaxterh commented Feb 8, 2026

Copy link
Copy Markdown
Owner

Fixes #1

Changes

  • Add \getMp4FallbackUrl()\ helper to derive MP4 URL from HLS URL
  • Add HLS error recovery for network and media errors
  • Fall back to direct MP4 (\play_360p.mp4) when HLS is unrecoverable
  • Handle video error events as additional fallback trigger

Testing

  1. Deploy to preview
  2. Test videos that only have 360p resolution (P-Rod Proof, The Coalition)
  3. Verify playback works via MP4 fallback

- Derive MP4 URL from HLS URL (playlist.m3u8 -> play_360p.mp4)
- Add error recovery for network and media errors
- Fall back to direct MP4 when HLS is unrecoverable
- Fixes playback issues with 360p-only Bunny streams
@cursor

cursor Bot commented Feb 8, 2026

Copy link
Copy Markdown

You have run out of free Bugbot PR reviews for this billing cycle. This will reset on March 11.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

@wbaxterh

Copy link
Copy Markdown
Owner Author

@copilot resolve the merge conflicts in this pull request

…fallback

# Conflicts:
#	components/media/VideoPlayer.js
#	styles/global.css

Copilot AI commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

Merge conflicts resolved in commit e41c650. Two files had conflicts:

  • components/media/VideoPlayer.js — kept the MP4 fallback logic from this PR while adopting the code style improvements (2-space indentation, _event prefix, single-quote strings) from main
  • styles/global.css — took main's version: removes !important from mobile nav colors and adds proper visibility: visible fixes for the navbar collapse

Build passes cleanly after the merge.

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.

Video player fails on 360p-only HLS streams

2 participants