Skip to content

[HLSTree] Set period duration also with live stream#1793

Merged
CastagnaIT merged 1 commit intoxbmc:Piersfrom
CastagnaIT:hls_period_dur
Mar 27, 2025
Merged

[HLSTree] Set period duration also with live stream#1793
CastagnaIT merged 1 commit intoxbmc:Piersfrom
CastagnaIT:hls_period_dur

Conversation

@CastagnaIT
Copy link
Copy Markdown
Collaborator

Description

on HLS live manifests that starts with a single period (or have), set the period duration to allow the video seek timeshift

this "limitation" come from old rework
but later has been merged a fix PR #1616 that has added a period check
that should prevent cause side effects at least on period change

Motivation and context

fix #1779

How has this been tested?

tested Raiplay addon (single period)
tested pluto tv (multi-periods)

seem no regressions

Screenshots (if appropriate):

Types of change

  • Bug fix (non-breaking change which fixes an issue)
  • Clean up (non-breaking change which removes non-working, unmaintained functionality)
  • Improvement (non-breaking change which improves existing functionality)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that will cause existing functionality to change)
  • Cosmetic change (non-breaking change that doesn't touch code)
  • None of the above (please explain below)

Checklist:

  • I have read the Contributing document
  • My code follows the Code Guidelines of this project
  • My change requires a change to the Wiki documentation
  • I have updated the documentation accordingly

with a single period manifest allow the video seek
@CastagnaIT CastagnaIT added Type: Fix non-breaking change which fixes an issue Component: HLS v22 Piers Backport: Done labels Mar 25, 2025
@CastagnaIT CastagnaIT merged commit 1306488 into xbmc:Piers Mar 27, 2025
9 checks passed
@CastagnaIT CastagnaIT deleted the hls_period_dur branch March 27, 2025 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Backport: Done Component: HLS Type: Fix non-breaking change which fixes an issue v22 Piers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Live M3U8 stream with widevine encryption is not seekable while it should be

1 participant