feat: retry logic for 412 not playable errors by Pintorado · Pull Request #1106 · muxinc/elements

Pintorado · 2025-04-28T20:02:52Z

Closes #531

This PR refactors how the player handles 412 (NETWORK_NOT_READY) errors.

Changes

Behavior:

Error dialog says “Live stream/video is not currently available. Retrying in [xx] seconds…” (different message depending if it's a live stream or VOD).
Retries after 5s the first time and then the retry time increases to 60s. It retries 5 more times with a 60s gap.
Stops retrying after 5 minutes and 5s and if there was no success on recovering the live stream/video, the UI shows "Live stream/video is not currently available. Try again later or click here to retry” with a retry link.

vercel · 2025-04-28T20:02:58Z

@Pintorado is attempting to deploy a commit to the Mux Team on Vercel.

A member of the Team first needs to authorize it.

snyk-io · 2025-04-28T20:03:05Z

🎉 Snyk checks have passed. No issues have been found so far.

✅ security/snyk check is complete. No issues have been found. (View Details)

✅ license/snyk check is complete. No issues have been found. (View Details)

packages/playback-core/src/errors.ts

packages/mux-player/src/errors.ts

luwes · 2025-06-03T18:57:58Z

packages/playback-core/src/index.ts

+            } else {
+              hls.recoverMediaError();
+            }


I think this shouldn't be used here. recoverMediaError() is an attempt to recover a media decode error is my understanding or at least it's definitely not for a network error.

You're right, I'll remove it

packages/playback-core/src/index.ts

packages/playback-core/src/request-errors.ts

packages/playback-core/src/index.ts

vercel · 2025-06-06T17:46:38Z

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
elements-demo-astro	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 16, 2025 2:26pm
elements-demo-nextjs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 16, 2025 2:26pm
elements-demo-svelte-kit	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 16, 2025 2:26pm
elements-demo-vanilla	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 16, 2025 2:26pm
elements-demo-vue	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 16, 2025 2:26pm

luwes

LGTM

🤖 I have created a release *beep* *boop* --- <details><summary>@mux/mux-audio: 0.15.10</summary> ## [0.15.10](https://github.com/muxinc/elements/compare/@mux/mux-audio@0.15.9...@mux/mux-audio@0.15.10) (2025-06-23) ### Dependencies * The following workspace dependencies were updated * dependencies * @mux/playback-core bumped from 0.29.1 to 0.30.0 </details> <details><summary>@mux/mux-audio-react: 0.15.10</summary> ## [0.15.10](https://github.com/muxinc/elements/compare/@mux/mux-audio-react@0.15.9...@mux/mux-audio-react@0.15.10) (2025-06-23) ### Dependencies * The following workspace dependencies were updated * dependencies * @mux/playback-core bumped from 0.29.1 to 0.30.0 </details> <details><summary>@mux/playback-core: 0.30.0</summary> ## [0.30.0](https://github.com/muxinc/elements/compare/@mux/playback-core@0.29.1...@mux/playback-core@0.30.0) (2025-06-23) ### Features * add free plan logo ([#1140](#1140)) ([a6b369e](a6b369e)) * Google IMA support for mux-player and mux-video variants ([#1128](#1128)) ([ec31d4d](ec31d4d)) * retry logic for 412 not playable errors ([#1106](#1106)) ([677c90a](677c90a)) ### Bug Fixes * **playback-core:** prefer uri value for chapters session metadata when available. ([#1145](#1145)) ([4dd2a50](4dd2a50)) * support URI and VALUE for session data ([#1146](#1146)) ([930ad72](930ad72)) </details> <details><summary>@mux/mux-player: 3.5.0</summary> ## [3.5.0](https://github.com/muxinc/elements/compare/@mux/mux-player@3.4.1...@mux/mux-player@3.5.0) (2025-06-23) ### Features * Google IMA support for mux-player and mux-video variants ([#1128](#1128)) ([ec31d4d](ec31d4d)) * retry logic for 412 not playable errors ([#1106](#1106)) ([677c90a](677c90a)) ### Dependencies * The following workspace dependencies were updated * dependencies * @mux/mux-video bumped from 0.25.3 to 0.26.0 * @mux/playback-core bumped from 0.29.1 to 0.30.0 </details> <details><summary>@mux/mux-player-react: 3.5.0</summary> ## [3.5.0](https://github.com/muxinc/elements/compare/@mux/mux-player-react@3.4.1...@mux/mux-player-react@3.5.0) (2025-06-23) ### Features * Google IMA support for mux-player and mux-video variants ([#1128](#1128)) ([ec31d4d](ec31d4d)) ### Bug Fixes * **mux-player-react news:** on demand ad tag url ([#1144](#1144)) ([fbf1bb0](fbf1bb0)) ### Dependencies * The following workspace dependencies were updated * dependencies * @mux/mux-player bumped from 3.4.1 to 3.5.0 * @mux/playback-core bumped from 0.29.1 to 0.30.0 </details> <details><summary>@mux/mux-video: 0.26.0</summary> ## [0.26.0](https://github.com/muxinc/elements/compare/@mux/mux-video@0.25.3...@mux/mux-video@0.26.0) (2025-06-23) ### Features * add free plan logo ([#1140](#1140)) ([a6b369e](a6b369e)) * Google IMA support for mux-player and mux-video variants ([#1128](#1128)) ([ec31d4d](ec31d4d)) ### Bug Fixes * typeerror getLogoHTML of undefined ([35478ce](35478ce)) ### Dependencies * The following workspace dependencies were updated * dependencies * @mux/playback-core bumped from 0.29.1 to 0.30.0 </details> <details><summary>@mux/mux-video-react: 0.26.0</summary> ## [0.26.0](https://github.com/muxinc/elements/compare/@mux/mux-video-react@0.25.3...@mux/mux-video-react@0.26.0) (2025-06-23) ### Miscellaneous Chores * **@mux/mux-video-react:** Synchronize video versions ### Dependencies * The following workspace dependencies were updated * dependencies * @mux/playback-core bumped from 0.29.1 to 0.30.0 </details> --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

retry logic for 412 errors

f63854d

ronalduQualabs self-requested a review April 30, 2025 15:06

ronalduQualabs approved these changes Apr 30, 2025

View reviewed changes

change max retries to constant

832bc86

Pintorado marked this pull request as ready for review April 30, 2025 16:47

Pintorado requested a review from a team as a code owner April 30, 2025 16:47

Pintorado added 2 commits May 14, 2025 11:49

Merge branch 'main' into retry-on-412-not-playable

514b4d8

Live stream vs video specific error messaging for NETWORK_NOT_READY

33c2e12

luwes reviewed May 14, 2025

View reviewed changes

packages/playback-core/src/errors.ts Show resolved Hide resolved

update retry logic for NETWORK_NOT_READY errors

9d98d96

Pintorado requested a review from luwes May 27, 2025 15:48