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 #1380, DVR: use publish param instead of play #2550

Merged
merged 2 commits into from
Oct 12, 2021

Conversation

duiniuluantanqin
Copy link
Member

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Aug 24, 2021

Codecov Report

Merging #2550 (7ed5b1d) into develop (93cd025) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #2550      +/-   ##
===========================================
- Coverage    59.90%   59.90%   -0.01%     
===========================================
  Files          121      121              
  Lines        50907    50908       +1     
===========================================
  Hits         30496    30496              
- Misses       20411    20412       +1     

| Impacted Files | Coverage Δ | |'

Translated to English while maintaining the markdown structure:

'| Impacted Files | Coverage Δ | |
|---|---|---|
| trunk/src/app/srs_app_dvr.cpp | 0.00% <0.00%> (ø) | |

'

Translated to English while maintaining the markdown structure:

'| trunk/src/app/srs_app_dvr.cpp | 0.00% <0.00%> (ø) | |


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data'

Translated to English while maintaining the markdown structure:

'> Δ = absolute <relative> (impact), ø = not affected, ? = missing data

Powered by Codecov. Last update 93cd025...7ed5b1d. Read the comment docs.

TRANS_BY_GPT3

@winlinvip winlinvip changed the title on_dvr: use publish param instead of play DVR: use publish param instead of play Aug 26, 2021
@winlinvip winlinvip removed their request for review August 26, 2021 23:41
@winlinvip winlinvip added this to the SRS 4.0 release milestone Aug 27, 2021
@duiniuluantanqin duiniuluantanqin changed the title DVR: use publish param instead of play fix #1380, DVR: use publish param instead of play Sep 6, 2021
@winlinvip winlinvip merged commit 579b90d into ossrs:develop Oct 12, 2021
winlinvip pushed a commit to winlinvip/srs that referenced this pull request Oct 12, 2021
@winlinvip winlinvip added the TransByAI Translated by AI/GPT. label Jul 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TransByAI Translated by AI/GPT.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

It seems that the URI carried in the 'param' when saving on_dvr may be mixed up.
4 participants