mbox series

[FFmpeg-devel,v3,0/2] HTTP Retry-After Support

Message ID 20240425142330.288815-1-derek.buitenhuis@gmail.com
Headers show
Series HTTP Retry-After Support | expand

Message

Derek Buitenhuis April 25, 2024, 2:23 p.m. UTC 
Changes since last set:
  * Updated commit message with RFC references.
  * Properly support Retry-After as both a date and integer number of seconds.

I have tested this against both an HTTP-Date and seconds, and confirmed
it to work.

Derek Buitenhuis (2):
  avformat/http: Rename parse_set_cookie_expiry_time to parse_http_date
  avformat/http: Add support for Retry-After header

 doc/protocols.texi    |  5 ++++
 libavformat/http.c    | 62 ++++++++++++++++++++++++++++++-------------
 libavformat/version.h |  2 +-
 3 files changed, 49 insertions(+), 20 deletions(-)

Comments

Martin Storsjö April 25, 2024, 8:22 p.m. UTC | #1 
On Thu, 25 Apr 2024, Derek Buitenhuis wrote:

> Changes since last set:
>  * Updated commit message with RFC references.
>  * Properly support Retry-After as both a date and integer number of seconds.
>
> I have tested this against both an HTTP-Date and seconds, and confirmed
> it to work.
>
> Derek Buitenhuis (2):
>  avformat/http: Rename parse_set_cookie_expiry_time to parse_http_date
>  avformat/http: Add support for Retry-After header
>
> doc/protocols.texi    |  5 ++++
> libavformat/http.c    | 62 ++++++++++++++++++++++++++++++-------------
> libavformat/version.h |  2 +-
> 3 files changed, 49 insertions(+), 20 deletions(-)

Thanks, these patches LGTM.

// Martin
Derek Buitenhuis April 26, 2024, 12:01 p.m. UTC | #2 
On 4/25/2024 9:22 PM, Martin Storsjö wrote:
> Thanks, these patches LGTM.

Pushed, thanks.

- Derek