Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(MSS): Fix playback of some MSS streams #7517

Merged
merged 1 commit into from
Oct 29, 2024

Conversation

avelad
Copy link
Member

@avelad avelad commented Oct 28, 2024

With this change, no default flags are used. It seems that when a sample has to use the default flags, in Safari, it can cause problems with MSS. With HLS this does not happen.

Fixes #7509

@avelad avelad added type: bug Something isn't working correctly priority: P2 Smaller impact or easy workaround browser: Safari Issues affecting Safari or WebKit derivatives labels Oct 28, 2024
@avelad avelad added this to the v4.12 milestone Oct 28, 2024
@avelad avelad added the component: MSS The issue involves Microsoft Smooth Streaming manifest format label Oct 28, 2024
@avelad
Copy link
Member Author

avelad commented Oct 28, 2024

@shaka-bot test

@shaka-bot
Copy link
Collaborator

@avelad: Lab tests started with arguments:

  • pr=7517

@shaka-bot
Copy link
Collaborator

Incremental code coverage: 100.00%

Copy link
Member

@joeyparrish joeyparrish left a comment

Choose a reason for hiding this comment

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

Could you add a small amount of detail in the PR description about these flags, what they were, what they are now, and why?

@avelad avelad merged commit 6d0ee51 into shaka-project:main Oct 29, 2024
36 of 37 checks passed
@avelad avelad deleted the mss-safari branch October 29, 2024 08:25
ncocaign added a commit to ncocaign/shaka-player that referenced this pull request Oct 29, 2024
fix(DASH): Fix playback of some DASH with multiple period (shaka-project#7516)

With this change, closeSegmentIndex() of stream removed of a period are defered in StreamingEngine.onUpdate_()

Fixes shaka-project#7517
ncocaign added a commit to ncocaign/shaka-player that referenced this pull request Oct 29, 2024
…ect#7517)

With this change, closeSegmentIndex() of all streams of a removed period are defered in StreamingEngine.onUpdate_()

Fixes shaka-project#7517
avelad added a commit that referenced this pull request Nov 12, 2024
With this change, no default flags are used. It seems that when a sample
has to use the default flags, in Safari, it can cause problems with MSS.
With HLS this does not happen.

Fixes #7509
avelad added a commit that referenced this pull request Nov 12, 2024
With this change, no default flags are used. It seems that when a sample
has to use the default flags, in Safari, it can cause problems with MSS.
With HLS this does not happen.

Fixes #7509
joeyparrish pushed a commit that referenced this pull request Nov 12, 2024
With this change, no default flags are used. It seems that when a sample
has to use the default flags, in Safari, it can cause problems with MSS.
With HLS this does not happen.

Fixes #7509
@shaka-bot shaka-bot added the status: archived Archived and locked; will not be updated label Dec 28, 2024
@shaka-project shaka-project locked as resolved and limited conversation to collaborators Dec 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
browser: Safari Issues affecting Safari or WebKit derivatives component: MSS The issue involves Microsoft Smooth Streaming manifest format priority: P2 Smaller impact or easy workaround status: archived Archived and locked; will not be updated type: bug Something isn't working correctly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Azure Media Service MSS doesn't work on iOS & Mac Safari
3 participants