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

RTC: fix play crash or no stream for rtmp2rtc tips.(#2728) #2863

Merged

Conversation

chundonglinlin
Copy link
Member

@chundonglinlin chundonglinlin commented Jan 10, 2022

If I don't publish stream, when I play RTC stream and SRS will crash. And if I publish RTC stream, when I play RTC stream and SRS don't distribute RTC stream.Config as follows:

vhost __defaultVhost__ {
    rtc {
        enabled     on;
        # @see https://github.com/ossrs/srs/wiki/v4_CN_WebRTC#rtmp-to-rtc
        rtmp_to_rtc off;
        # @see https://github.com/ossrs/srs/wiki/v4_CN_WebRTC#rtc-to-rtmp
        rtc_to_rtmp off;
    }
}

@codecov-commenter
Copy link

codecov-commenter commented Jan 10, 2022

Codecov Report

Merging #2863 (f5b601d) into 4.0release (08910e5) will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           4.0release    #2863   +/-   ##
===========================================
  Coverage       60.22%   60.22%           
===========================================
  Files             121      121           
  Lines           51070    51070           
===========================================
  Hits            30756    30756           
  Misses          20314    20314           

| Impacted Files | Coverage Δ | |'

Translated to English while maintaining the markdown structure:

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

Translated to English while maintaining the markdown structure:

| trunk/src/app/srs_app_rtc_api.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 08910e5...f5b601d. Read the comment docs.

TRANS_BY_GPT3

@winlinvip winlinvip merged commit c85daeb into ossrs:4.0release Jan 10, 2022
chundonglinlin added a commit to chundonglinlin/srs that referenced this pull request Jan 10, 2022
chundonglinlin added a commit to chundonglinlin/srs that referenced this pull request Jan 11, 2022
chundonglinlin added a commit to chundonglinlin/srs that referenced this pull request Jan 12, 2022
chundonglinlin added a commit to chundonglinlin/srs that referenced this pull request Jan 12, 2022
chundonglinlin pushed a commit to chundonglinlin/srs that referenced this pull request Jan 13, 2022
chundonglinlin added a commit to chundonglinlin/srs that referenced this pull request Jan 13, 2022
@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.

3 participants