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.

Sign up for GitHub

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

Jump to bottom

For #2371: Add regression test for SDP nack #2487

Merged
merged 2 commits into from
Jul 24, 2021
Merged

For #2371: Add regression test for SDP nack #2487

merged 2 commits into from
Jul 24, 2021

Conversation

winlinvip
Copy link
Member

  1. Refine API to change defaule decodes.
  2. Add test for publish SDP nack.
  3. Add test for play SDP nack.

@winlinvip winlinvip linked an issue Jul 24, 2021 that may be closed by this pull request
RTMP streaming with WebRTC playback, works fine on Chrome but experiences packet loss and buffering on Firefox. #2371
Closed
duiniuluantanqin and others added 2 commits July 24, 2021 20:49
@duiniuluantanqin @winlinvip
explicitly enable nack, for firefox (ossrs#2373)
43bb966
@winlinvip
For ossrs#2371: Add regression test for SDP nack
400e3ad 
1. Refine API to change defaule decodes.
2. Add test for publish SDP nack.
3. Add test for play SDP nack.
@winlinvip winlinvip force-pushed the bugfix/test_for_nack_verify branch from a9930cb to 400e3ad Compare July 24, 2021 13:00
@winlinvip winlinvip merged commit 8c7ba05 into ossrs:4.0release Jul 24, 2021
@winlinvip winlinvip added the EnglishNative This issue is conveyed exclusively in English. label Jul 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@duiniuluantanqin duiniuluantanqin Awaiting requested review from duiniuluantanqin

Assignees
No one assigned
Labels
EnglishNative This issue is conveyed exclusively in English.
Projects
None yet
Milestone
No milestone
2 participants
@winlinvip @duiniuluantanqin