http-streaming icon indicating copy to clipboard operation
http-streaming copied to clipboard

feat: streaming events and errors

Open adrums86 opened this issue 1 year ago • 1 comments

Description

Adding some additional events and errors with relevant payloads to VHS to enable users to centralize streaming events and utilize the information passed with each error and event payload to inform their video playback code.

Specific Changes proposed

Add events and payloads for all phases of playback from manifest/playlist request - load - parse. Segment load - transmux/parse/decrypt - append. Errors and relevant payloads for common network, segment and playlist/manifest conditions.

Requirements Checklist

  • [x] Feature implemented / Bug fixed
  • [x] If necessary, more likely in a feature request than a bug fix
  • [ ] Reviewed by Two Core Contributors

adrums86 avatar May 05 '24 23:05 adrums86

Codecov Report

Attention: Patch coverage is 88.68778% with 25 lines in your changes are missing coverage. Please review.

Project coverage is 86.31%. Comparing base (b6ff608) to head (a14d0e6).

Files Patch % Lines
src/dash-playlist-loader.js 80.76% 5 Missing :warning:
src/media-segment-request.js 87.17% 5 Missing :warning:
src/source-updater.js 37.50% 5 Missing :warning:
src/content-steering-controller.js 77.77% 2 Missing :warning:
src/playlist-loader.js 93.10% 2 Missing :warning:
src/segment-loader.js 94.73% 2 Missing :warning:
src/segment-transmuxer.js 60.00% 2 Missing :warning:
src/playlist-controller.js 95.23% 1 Missing :warning:
src/videojs-http-streaming.js 90.90% 1 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1508      +/-   ##
==========================================
+ Coverage   86.29%   86.31%   +0.01%     
==========================================
  Files          43       43              
  Lines       10901    11098     +197     
  Branches     2508     2532      +24     
==========================================
+ Hits         9407     9579     +172     
- Misses       1494     1519      +25     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar May 10 '24 16:05 codecov[bot]