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

Edge: fix origin fail for missing or incomplete params. #4306

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

chundonglinlin
Copy link
Member

@chundonglinlin chundonglinlin commented Mar 18, 2025

1. Background

When setting up edge nodes for origin retrieval mode, sometimes the origin server requires authentication parameters for validation. If a playback request reaches the edge and triggers origin retrieval without these parameters, it can result in playback failure.

2. Configuration

Configuration file:

listen              1935;
max_connections     1000;
pid                 objs/edge.pid;
daemon              off;
srs_log_tank        console;
http_server {
    enabled         on;
    listen          8080;
    dir             ./objs/nginx/html;
}
vhost __defaultVhost__ {
    cluster {
        mode            remote;
        origin          127.0.0.1:19350;
    }
    http_remux {
        enabled     on;
        mount       [vhost]/[app]/[stream].flv;
    }
}

3. Playback Request

Format 1

ffplay -v debug "rtmp://127.0.0.1/live?secret=84af974ae64b2e1d&time=3C5DA3FA7/livestream"

Format 2

ffplay -v debug "rtmp://127.0.0.1/live?secret=84af974ae64b2e1d/livestream?time=3C5DA3FA7"

Format 3

ffplay -v debug "rtmp://127.0.0.1/live/livestream?secret=84af974ae64b2e1d&time=3C5DA3FA7"

TRANS_BY_GPT4

@winlinvip winlinvip added the TransByAI Translated by AI/GPT. label Mar 18, 2025
@duiniuluantanqin
Copy link
Member

duiniuluantanqin commented Mar 24, 2025

Add some unit tests.

TRANS_BY_GPT4

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