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

When the ts file name contains [duration], the on_hls_notify callback address is generated incorrectly. #1778

Closed
lileiseven opened this issue May 24, 2020 · 1 comment · Fixed by #2549
Assignees
Labels
Bug It might be a bug. TransByAI Translated by AI/GPT.
Milestone

Comments

@lileiseven
Copy link

lileiseven commented May 24, 2020

Configuration example:

vhost __defaultVhost__ {
    http_hooks {
        enabled         on;
        on_hls_notify   http://cdn.domain/[app]/[stream]/[ts_url][param];
    }
    hls {
        enabled         on;
        hls_fragment    4;
        hls_window      12;
        hls_path        ./objs/nginx/html;
        hls_m3u8_file   [app]/[stream].m3u8;
        hls_ts_file     [app]/[stream]-[seq]-[timestamp]-[duration].ts;
    }
}

The logs seen on the CDN side are:

***.***.***.*** - - [25/May/2020:01:30:05 +0800] "GET /xxx/xxxx/xxxx-146-1590341405159-[duration].ts HTTP/1.1" 404 169 "-" "SRS/3.0.123(OuXuli)"

The placeholder [duration] has not been replaced.

TRANS_BY_GPT3

@winlinvip
Copy link
Member

Check it

@winlinvip winlinvip added the Bug It might be a bug. label Dec 1, 2020
@winlinvip winlinvip added this to the SRS 4.0 release milestone Aug 26, 2021
@duiniuluantanqin duiniuluantanqin linked a pull request Aug 26, 2021 that will close this issue
@winlinvip winlinvip changed the title ts文件名中带[duration]时,on_hls_notify 回调地址生成错误 When the ts file name contains [duration], the on_hls_notify callback address is generated incorrectly. Jul 29, 2023
@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
Bug It might be a bug. TransByAI Translated by AI/GPT.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants