diff mbox

[FFmpeg-devel] avformat/dashdec: propagate icy to child AVIOContexts

Message ID 20191217004504.29011-1-epirat07@gmail.com
State Accepted
Commit f37bfd3f62899d5aa1460b0d206a32a602e1c9ff
Headers show

Commit Message

Marvin Scholz Dec. 17, 2019, 12:45 a.m. UTC
When the user decides they do not want to to send the Icy-MetaData
header, this should be respected for all requests, not just the first
one.

Fix #5578
---
 libavformat/dashdec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Marvin Scholz Dec. 21, 2019, 11:03 a.m. UTC | #1
On 17 Dec 2019, at 1:45, Marvin Scholz wrote:

> When the user decides they do not want to to send the Icy-MetaData
> header, this should be respected for all requests, not just the first
> one.
>
> Fix #5578
> ---
>  libavformat/dashdec.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavformat/dashdec.c b/libavformat/dashdec.c
> index 72ba9605f0..15e79fd51a 100644
> --- a/libavformat/dashdec.c
> +++ b/libavformat/dashdec.c
> @@ -1851,7 +1851,7 @@ static int save_avio_options(AVFormatContext *s)
>  {
>      DASHContext *c = s->priv_data;
>      const char *opts[] = {
> -        "headers", "user_agent", "cookies", "http_proxy", "referer", 
> "rw_timeout", NULL };
> +        "headers", "user_agent", "cookies", "http_proxy", "referer", 
> "rw_timeout", "icy", NULL };
>      const char **opt = opts;
>      uint8_t *buf = NULL;
>      int ret = 0;
> -- 
> 2.20.1 (Apple Git-117)

ping for review, please :)
Liu Steven Dec. 21, 2019, 11:11 p.m. UTC | #2
> 在 2019年12月21日,下午7:03,Marvin Scholz <epirat07@gmail.com> 写道:
> 
> On 17 Dec 2019, at 1:45, Marvin Scholz wrote:
> 
>> When the user decides they do not want to to send the Icy-MetaData
>> header, this should be respected for all requests, not just the first
>> one.
>> 
>> Fix #5578
>> ---
>> libavformat/dashdec.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>> 
>> diff --git a/libavformat/dashdec.c b/libavformat/dashdec.c
>> index 72ba9605f0..15e79fd51a 100644
>> --- a/libavformat/dashdec.c
>> +++ b/libavformat/dashdec.c
>> @@ -1851,7 +1851,7 @@ static int save_avio_options(AVFormatContext *s)
>> {
>>     DASHContext *c = s->priv_data;
>>     const char *opts[] = {
>> -        "headers", "user_agent", "cookies", "http_proxy", "referer", "rw_timeout", NULL };
>> +        "headers", "user_agent", "cookies", "http_proxy", "referer", "rw_timeout", "icy", NULL };
>>     const char **opt = opts;
>>     uint8_t *buf = NULL;
>>     int ret = 0;
>> -- 
>> 2.20.1 (Apple Git-117)
> 
> ping for review, please :)

LGTM
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

Steven
Thanks
James Almer Dec. 27, 2019, 7:28 p.m. UTC | #3
On 12/21/2019 8:11 PM, Liu Steven wrote:
> 
> 
>> 在 2019年12月21日,下午7:03,Marvin Scholz <epirat07@gmail.com> 写道:
>>
>> On 17 Dec 2019, at 1:45, Marvin Scholz wrote:
>>
>>> When the user decides they do not want to to send the Icy-MetaData
>>> header, this should be respected for all requests, not just the first
>>> one.
>>>
>>> Fix #5578
>>> ---
>>> libavformat/dashdec.c | 2 +-
>>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/libavformat/dashdec.c b/libavformat/dashdec.c
>>> index 72ba9605f0..15e79fd51a 100644
>>> --- a/libavformat/dashdec.c
>>> +++ b/libavformat/dashdec.c
>>> @@ -1851,7 +1851,7 @@ static int save_avio_options(AVFormatContext *s)
>>> {
>>>     DASHContext *c = s->priv_data;
>>>     const char *opts[] = {
>>> -        "headers", "user_agent", "cookies", "http_proxy", "referer", "rw_timeout", NULL };
>>> +        "headers", "user_agent", "cookies", "http_proxy", "referer", "rw_timeout", "icy", NULL };
>>>     const char **opt = opts;
>>>     uint8_t *buf = NULL;
>>>     int ret = 0;
>>> -- 
>>> 2.20.1 (Apple Git-117)
>>
>> ping for review, please :)
> 
> LGTM

Pushed.
diff mbox

Patch

diff --git a/libavformat/dashdec.c b/libavformat/dashdec.c
index 72ba9605f0..15e79fd51a 100644
--- a/libavformat/dashdec.c
+++ b/libavformat/dashdec.c
@@ -1851,7 +1851,7 @@  static int save_avio_options(AVFormatContext *s)
 {
     DASHContext *c = s->priv_data;
     const char *opts[] = {
-        "headers", "user_agent", "cookies", "http_proxy", "referer", "rw_timeout", NULL };
+        "headers", "user_agent", "cookies", "http_proxy", "referer", "rw_timeout", "icy", NULL };
     const char **opt = opts;
     uint8_t *buf = NULL;
     int ret = 0;