From patchwork Sat Sep 18 23:05:43 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jordi Cenzano X-Patchwork-Id: 30366 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a6b:6506:0:0:0:0:0 with SMTP id z6csp240744iob; Sat, 18 Sep 2021 16:06:08 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyfBg6vtave2qCbw3nubiNMqnI080lUsD0/xd6GPg01m1miX52p+UHO1rrvVjLd0gbwL7zp X-Received: by 2002:aa7:c85a:: with SMTP id g26mr20678095edt.217.1632006368077; Sat, 18 Sep 2021 16:06:08 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1632006368; cv=none; d=google.com; s=arc-20160816; b=tm31T/+ZqL4iEDEL3YxFiei+hIoBExh20ClIJNHtITWBqZmaLqCB9r+H4rG7a8egPJ Eh8liqVmH29IXNdgfj0Itf4cIj9nE0fSdoYCCBAFkAjvFDNVua4W6SSA9bS2xGgUGDRR RJxkrqXI/FsglDTiwRq7WLq/eeJy715s+5bkkFhjkVPXR0B6NkQCJ/NMSHQONFteKoMM 0b8u52R8p6EFEOrZPQeN/6rEg4oB9cjbl6z1oFMoqvY04oIRG8C50Hyp8ikH/n8g+cCD o2ca6YkkyxKlHz/IJ26E9BCVD0VqLQMJ3Ao/UIYrEsGyhUgqEG2Mt55/SqLw097DJ8WQ GRig== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:content-transfer-encoding:cc:reply-to :list-subscribe:list-help:list-post:list-archive:list-unsubscribe :list-id:precedence:subject:mime-version:message-id:date:to:from :dkim-signature:delivered-to; bh=hZsGqPSE/lhhgsf0/Y6BpnfFLMh/ijeEqA1vt4nCyTM=; b=o1SDHkRYsWwxGs6NDbsccCfvY9hkUFEodjHM3GBYfbmAR8MGpJ8plAkS3ZT1MuWrnm yTTIU9FPMYtJR+R/MlElient86Ps1eSCUOtkwBx7PGs4CVWVeXxCNBtxYBzLEZhjuZDI PZoxEmOBbqTNPh59kABM0C/SXbDuWr0ebaZ1ub1YDglpuB4WuxD/4GycCkVcxKPRQ5lD 8b4qv4LXlU4OGwRxsBeKyCxBWAzWGQmgGrvjlhURf32c3UW22Fe5aMcOTvUH8NaER3Md htXH/MwGGUdXufbH8DDi8OJo2wZix2+6/DIeFzPU0gZycCFvG7jytRsUHQi7dXOoDSkk p7ag== ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@gmail.com header.s=20210112 header.b=F8NT8QXT; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org. [79.124.17.100]) by mx.google.com with ESMTP id my5si10949397ejc.208.2021.09.18.16.06.01; Sat, 18 Sep 2021 16:06:08 -0700 (PDT) Received-SPF: pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) client-ip=79.124.17.100; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@gmail.com header.s=20210112 header.b=F8NT8QXT; spf=pass (google.com: domain of ffmpeg-devel-bounces@ffmpeg.org designates 79.124.17.100 as permitted sender) smtp.mailfrom=ffmpeg-devel-bounces@ffmpeg.org; dmarc=fail (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 34D9868AA8A; Sun, 19 Sep 2021 02:05:58 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mail-wm1-f54.google.com (mail-wm1-f54.google.com [209.85.128.54]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 15E8768AA8A for ; Sun, 19 Sep 2021 02:05:51 +0300 (EEST) Received: by mail-wm1-f54.google.com with SMTP id u19-20020a7bc053000000b002f8d045b2caso9395453wmc.1 for ; Sat, 18 Sep 2021 16:05:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=0qxJCx1ETbFJjxl+d3rZrt4ga9QFY9ojmDrUo4eXPnM=; b=F8NT8QXTKceSGFKkW+xEvkdcz/VN7uvlBx8Kl8mxrjJCvuiGSdHHzIrzlh+sGHOT0V rWYtfmM3d48w5TmI6cMS6isZpFROb8y22q2n1BfuDxpZC5fA4OvqQXOlv005c8iSbR1u xmppdGFTBZr3ftcSYwJUK1VCSGjl6taKenTtjNZlszPzWI7AKfj+VgwZSLub09FWu5hB kZFzCwfUd7PY/XKGJBg4LcDoPPJ6zD1UvMZJ2u1dRzMPLqQADCYDrl4Q/wZ+BzDD/hGa bXfVLSIf1iXQg03zD7lWuJGnMI1t1LsRu6o739KrdPEZ0tgy1YPwS/2fXbj0EBa+KC0a zuEA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=0qxJCx1ETbFJjxl+d3rZrt4ga9QFY9ojmDrUo4eXPnM=; b=J9469xWdT8ODo0woGKwGs4LaFKrkGuaI3OP9ohT2xIfwWs5CU1CH30/B6Ox9VjGGpD PVyFsQ5lchUoFwrQv9CFDHrVKCe+pGDdTP1K7998TIqqHAVfCsfD3D49deyuwLOx6S/O oUkYqT1cqC+rJErx2mAVm9nvGVbfnXwYVCiX3iaG2dTR4eB/tV6K4Ap81BToc8wQ3iXl hVBJyIWXMtISswboqGy8nEWJlgXvxIYafBX0zLXozkZ8n/DyxArzGBQ7PHFUjfu/eMCx mDRkSo04S3jl+tNbCmYKF1UG+MM5c0godAmbeQqaziUjX2Xdzq8zFzrEDUeMTMqYTypk c9nA== X-Gm-Message-State: AOAM531xoLo30TycNN6ujCo2K1i9VRE7orgUHXGcjRzw39yC6juGmso/ EIWFjfByr1yAsNzkEUErTXEOjgNVzzAPwuYSlZQ= X-Received: by 2002:a1c:1dcd:: with SMTP id d196mr16667284wmd.9.1632006349668; Sat, 18 Sep 2021 16:05:49 -0700 (PDT) Received: from ip-172-31-15-150.eu-west-3.compute.internal (ec2-15-237-28-106.eu-west-3.compute.amazonaws.com. [15.237.28.106]) by smtp.gmail.com with ESMTPSA id v17sm10755833wrr.69.2021.09.18.16.05.49 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Sat, 18 Sep 2021 16:05:49 -0700 (PDT) From: Jordi Cenzano To: ffmpeg-devel@ffmpeg.org Date: Sat, 18 Sep 2021 23:05:43 +0000 Message-Id: <20210918230543.26860-1-jordi.cenzano@gmail.com> X-Mailer: git-send-email 2.32.0 MIME-Version: 1.0 Subject: [FFmpeg-devel] [PATCH 1/1] Implements RTMP reconnect feature X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Cc: Jordi Cenzano Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" X-TUID: WLF7Tt0/6Tc3 Nowadays when you are streaming to a live platform if the RTMP(s) server needs to restarted for any reason (ex: deploy new version) the RTMP connection is interrupted (probably after some draining time). Facebook will publish a proposal to avoid that by sending a GoAway message in the RTMP protocol. This code is the reference client implementation of that proposal. AFAIK other big live platforms showed their interest in implementing this mechanism. This can be already tested against Facebook live production using the querystring parameter ?ccr_sec=120 (that indicates the backend to send a disconnect signal after those seconds) --- libavformat/rtmppkt.c | 19 +++ libavformat/rtmppkt.h | 10 ++ libavformat/rtmpproto.c | 356 +++++++++++++++++++++++++++++++++++++--- 3 files changed, 359 insertions(+), 26 deletions(-) diff --git a/libavformat/rtmppkt.c b/libavformat/rtmppkt.c index 4b97c0833f..84ec72740d 100644 --- a/libavformat/rtmppkt.c +++ b/libavformat/rtmppkt.c @@ -405,6 +405,25 @@ int ff_rtmp_packet_write(URLContext *h, RTMPPacket *pkt, return written; } +int ff_rtmp_packet_clone(RTMPPacket *pkt_dst, const RTMPPacket *pkt_src) +{ + if (pkt_src->size) { + pkt_dst->data = av_realloc(NULL, pkt_src->size); + if (!pkt_dst->data) + return AVERROR(ENOMEM); + else + memcpy(pkt_dst->data, pkt_src->data, pkt_src->size); + } + pkt_dst->size = pkt_src->size; + pkt_dst->channel_id = pkt_src->channel_id; + pkt_dst->type = pkt_src->type; + pkt_dst->timestamp = pkt_src->timestamp; + pkt_dst->extra = pkt_src->extra; + pkt_dst->ts_field = pkt_src->ts_field; + + return 0; +} + int ff_rtmp_packet_create(RTMPPacket *pkt, int channel_id, RTMPPacketType type, int timestamp, int size) { diff --git a/libavformat/rtmppkt.h b/libavformat/rtmppkt.h index a15d2a5773..cdb901df89 100644 --- a/libavformat/rtmppkt.h +++ b/libavformat/rtmppkt.h @@ -59,6 +59,7 @@ typedef enum RTMPPacketType { RTMP_PT_SHARED_OBJ, ///< shared object RTMP_PT_INVOKE, ///< invoke some stream action RTMP_PT_METADATA = 22, ///< FLV metadata + RTMP_PT_GO_AWAY = 32, ///< Indicates please reconnect ASAP, server is about to go down } RTMPPacketType; /** @@ -99,6 +100,15 @@ typedef struct RTMPPacket { int ff_rtmp_packet_create(RTMPPacket *pkt, int channel_id, RTMPPacketType type, int timestamp, int size); +/** + * Clone RTMP packet + * + * @param pkt_dst packet destination + * @param pkt_src packet source + * @return zero on success, negative value otherwise + */ +int ff_rtmp_packet_clone(RTMPPacket *pkt_dst, const RTMPPacket *pkt_src); + /** * Free RTMP packet. * diff --git a/libavformat/rtmpproto.c b/libavformat/rtmpproto.c index b14d23b919..5b52c23f20 100644 --- a/libavformat/rtmpproto.c +++ b/libavformat/rtmpproto.c @@ -124,11 +124,21 @@ typedef struct RTMPContext { int nb_streamid; ///< The next stream id to return on createStream calls double duration; ///< Duration of the stream in seconds as returned by the server (only valid if non-zero) int tcp_nodelay; ///< Use TCP_NODELAY to disable Nagle's algorithm if set to 1 + int reconnect_interval; ///< Forces a reconnected every Xs (in media time) char username[50]; char password[50]; char auth_params[500]; int do_reconnect; + uint32_t last_reconnect_timestamp; int auth_tried; + int force_reconnection_now; + int go_away_received; + AVDictionary* original_opts; + char original_uri[TCURL_MAX_LENGTH]; + int original_flags; + RTMPPacket last_avc_seq_header_pkt; ///< rtmp packet, used to save last AVC video header, used on reconnection + RTMPPacket last_aac_seq_header_pkt; ///< rtmp packet, used to save last AAC audio header, used on reconnection + RTMPPacket last_metadata_pkt; ///< rtmp packet, used to save last onMetadata info, used on reconnection } RTMPContext; #define PLAYER_KEY_OPEN_PART_LEN 30 ///< length of partial key used for first client digest signing @@ -224,7 +234,7 @@ static void free_tracked_methods(RTMPContext *rt) rt->nb_tracked_methods = 0; } -static int rtmp_send_packet(RTMPContext *rt, RTMPPacket *pkt, int track) +static int rtmp_send_packet(RTMPContext *rt, RTMPPacket *pkt, int track, int destroy) { int ret; @@ -248,7 +258,9 @@ static int rtmp_send_packet(RTMPContext *rt, RTMPPacket *pkt, int track) ret = ff_rtmp_packet_write(rt->stream, pkt, rt->out_chunk_size, &rt->prev_pkt[1], &rt->nb_prev_pkt[1]); fail: - ff_rtmp_packet_destroy(pkt); + if (destroy) + ff_rtmp_packet_destroy(pkt); + return ret; } @@ -336,6 +348,9 @@ static int gen_connect(URLContext *s, RTMPContext *rt) if (!rt->is_input) { ff_amf_write_field_name(&p, "type"); ff_amf_write_string(&p, "nonprivate"); + // Indicates accepts goaway + ff_amf_write_field_name(&p, "goaway"); + ff_amf_write_bool(&p, 1); } ff_amf_write_field_name(&p, "flashVer"); ff_amf_write_string(&p, rt->flashver); @@ -400,7 +415,7 @@ static int gen_connect(URLContext *s, RTMPContext *rt) pkt.size = p - pkt.data; - return rtmp_send_packet(rt, &pkt, 1); + return rtmp_send_packet(rt, &pkt, 1, 1); } @@ -611,7 +626,7 @@ static int gen_release_stream(URLContext *s, RTMPContext *rt) ff_amf_write_null(&p); ff_amf_write_string(&p, rt->playpath); - return rtmp_send_packet(rt, &pkt, 1); + return rtmp_send_packet(rt, &pkt, 1, 1); } /** @@ -635,7 +650,7 @@ static int gen_fcpublish_stream(URLContext *s, RTMPContext *rt) ff_amf_write_null(&p); ff_amf_write_string(&p, rt->playpath); - return rtmp_send_packet(rt, &pkt, 1); + return rtmp_send_packet(rt, &pkt, 1, 1); } /** @@ -659,7 +674,7 @@ static int gen_fcunpublish_stream(URLContext *s, RTMPContext *rt) ff_amf_write_null(&p); ff_amf_write_string(&p, rt->playpath); - return rtmp_send_packet(rt, &pkt, 0); + return rtmp_send_packet(rt, &pkt, 0, 1); } /** @@ -683,7 +698,7 @@ static int gen_create_stream(URLContext *s, RTMPContext *rt) ff_amf_write_number(&p, ++rt->nb_invokes); ff_amf_write_null(&p); - return rtmp_send_packet(rt, &pkt, 1); + return rtmp_send_packet(rt, &pkt, 1, 1); } @@ -709,7 +724,7 @@ static int gen_delete_stream(URLContext *s, RTMPContext *rt) ff_amf_write_null(&p); ff_amf_write_number(&p, rt->stream_id); - return rtmp_send_packet(rt, &pkt, 0); + return rtmp_send_packet(rt, &pkt, 0, 1); } /** @@ -733,7 +748,7 @@ static int gen_get_stream_length(URLContext *s, RTMPContext *rt) ff_amf_write_null(&p); ff_amf_write_string(&p, rt->playpath); - return rtmp_send_packet(rt, &pkt, 1); + return rtmp_send_packet(rt, &pkt, 1, 1); } /** @@ -754,7 +769,7 @@ static int gen_buffer_time(URLContext *s, RTMPContext *rt) bytestream_put_be32(&p, rt->stream_id); bytestream_put_be32(&p, rt->client_buffer_time); - return rtmp_send_packet(rt, &pkt, 0); + return rtmp_send_packet(rt, &pkt, 0, 1); } /** @@ -782,7 +797,7 @@ static int gen_play(URLContext *s, RTMPContext *rt) ff_amf_write_string(&p, rt->playpath); ff_amf_write_number(&p, rt->live * 1000); - return rtmp_send_packet(rt, &pkt, 1); + return rtmp_send_packet(rt, &pkt, 1, 1); } static int gen_seek(URLContext *s, RTMPContext *rt, int64_t timestamp) @@ -805,7 +820,7 @@ static int gen_seek(URLContext *s, RTMPContext *rt, int64_t timestamp) ff_amf_write_null(&p); //as usual, the first null param ff_amf_write_number(&p, timestamp); //where we want to jump - return rtmp_send_packet(rt, &pkt, 1); + return rtmp_send_packet(rt, &pkt, 1, 1); } /** @@ -832,7 +847,7 @@ static int gen_pause(URLContext *s, RTMPContext *rt, int pause, uint32_t timesta ff_amf_write_bool(&p, pause); // pause or unpause ff_amf_write_number(&p, timestamp); //where we pause the stream - return rtmp_send_packet(rt, &pkt, 1); + return rtmp_send_packet(rt, &pkt, 1, 1); } /** @@ -859,7 +874,7 @@ static int gen_publish(URLContext *s, RTMPContext *rt) ff_amf_write_string(&p, rt->playpath); ff_amf_write_string(&p, "live"); - return rtmp_send_packet(rt, &pkt, 1); + return rtmp_send_packet(rt, &pkt, 1, 1); } /** @@ -885,7 +900,7 @@ static int gen_pong(URLContext *s, RTMPContext *rt, RTMPPacket *ppkt) bytestream_put_be16(&p, 7); // PingResponse bytestream_put_be32(&p, AV_RB32(ppkt->data+2)); - return rtmp_send_packet(rt, &pkt, 0); + return rtmp_send_packet(rt, &pkt, 0, 1); } /** @@ -906,7 +921,7 @@ static int gen_swf_verification(URLContext *s, RTMPContext *rt) bytestream_put_be16(&p, 27); memcpy(p, rt->swfverification, 42); - return rtmp_send_packet(rt, &pkt, 0); + return rtmp_send_packet(rt, &pkt, 0, 1); } /** @@ -925,7 +940,7 @@ static int gen_window_ack_size(URLContext *s, RTMPContext *rt) p = pkt.data; bytestream_put_be32(&p, rt->max_sent_unacked); - return rtmp_send_packet(rt, &pkt, 0); + return rtmp_send_packet(rt, &pkt, 0, 1); } /** @@ -946,7 +961,7 @@ static int gen_check_bw(URLContext *s, RTMPContext *rt) ff_amf_write_number(&p, ++rt->nb_invokes); ff_amf_write_null(&p); - return rtmp_send_packet(rt, &pkt, 1); + return rtmp_send_packet(rt, &pkt, 1, 1); } /** @@ -965,7 +980,7 @@ static int gen_bytes_read(URLContext *s, RTMPContext *rt, uint32_t ts) p = pkt.data; bytestream_put_be32(&p, rt->bytes_read); - return rtmp_send_packet(rt, &pkt, 0); + return rtmp_send_packet(rt, &pkt, 0, 1); } static int gen_fcsubscribe_stream(URLContext *s, RTMPContext *rt, @@ -985,7 +1000,7 @@ static int gen_fcsubscribe_stream(URLContext *s, RTMPContext *rt, ff_amf_write_null(&p); ff_amf_write_string(&p, subscribe); - return rtmp_send_packet(rt, &pkt, 1); + return rtmp_send_packet(rt, &pkt, 1, 1); } /** @@ -2153,6 +2168,16 @@ static int handle_invoke_status(URLContext *s, RTMPPacket *pkt) return 0; } +static int handle_go_away(URLContext *s, RTMPPacket *pkt) { + RTMPContext *rt = s->priv_data; + + av_log(s, AV_LOG_TRACE, "go away signal received"); + + rt->go_away_received = 1; + + return 0; +} + static int handle_invoke(URLContext *s, RTMPPacket *pkt) { RTMPContext *rt = s->priv_data; @@ -2331,6 +2356,10 @@ static int rtmp_parse_result(URLContext *s, RTMPContext *rt, RTMPPacket *pkt) if ((ret = handle_invoke(s, pkt)) < 0) return ret; break; + case RTMP_PT_GO_AWAY: + if ((ret = handle_go_away(s, pkt)) < 0) + return ret; + break; case RTMP_PT_VIDEO: case RTMP_PT_AUDIO: case RTMP_PT_METADATA: @@ -2513,6 +2542,15 @@ static int rtmp_close(URLContext *h) free_tracked_methods(rt); av_freep(&rt->flv_data); ffurl_closep(&rt->stream); + if (rt->last_avc_seq_header_pkt.size) + ff_rtmp_packet_destroy(&rt->last_avc_seq_header_pkt); + + if (rt->last_aac_seq_header_pkt.size) + ff_rtmp_packet_destroy(&rt->last_aac_seq_header_pkt); + + if (rt->last_metadata_pkt.size) + ff_rtmp_packet_destroy(&rt->last_metadata_pkt); + return ret; } @@ -2871,14 +2909,23 @@ reconnect: goto fail; } } else { - rt->flv_size = 0; - rt->flv_data = NULL; - rt->flv_off = 0; - rt->skip_bytes = 13; + // Do not clean buffers if it is a forced reconnection + if (rt->force_reconnection_now <= 0) { + rt->flv_size = 0; + rt->flv_data = NULL; + rt->flv_off = 0; + rt->skip_bytes = 13; + } } s->max_packet_size = rt->stream->max_packet_size; s->is_streamed = 1; + + // Copy original params + av_dict_copy(&rt->original_opts, *opts, 0); + rt->original_flags = flags; + av_strlcpy(rt->original_uri, uri, TCURL_MAX_LENGTH); + return 0; fail: @@ -2951,6 +2998,107 @@ static int rtmp_pause(URLContext *s, int pause) return 0; } +/** + * Reconnect RTMP connection. +*/ +static int rtmp_reconnect(URLContext *s) { + RTMPContext *rt = s->priv_data; + int i; + + // Close current RTMP connection + av_log(s, AV_LOG_INFO, "reconnecting!\n"); + + ffurl_closep(&rt->stream); + rt->do_reconnect = 0; + rt->nb_invokes = 0; + for (i = 0; i < 2; i++) + memset(rt->prev_pkt[i], 0, sizeof(**rt->prev_pkt) * rt->nb_prev_pkt[i]); + + free_tracked_methods(rt); + + // Connect RTMP again using orignal values + return rtmp_open(s, rt->original_uri, rt->original_flags, &rt->original_opts); +} + +/** + * Checks RTMP packet and return 1 when it contains an AAC header +*/ +static int rtmp_packet_is_aac_audio_header(RTMPPacket *pkt) { + uint8_t sound_format; + uint8_t aac_packet_type; + + if ((!pkt) || (pkt->size < 2) || pkt->type != RTMP_PT_AUDIO) + return 0; + + sound_format = (pkt->data[0] & 0xF0) >> 4; + aac_packet_type = pkt->data[1]; + // Check codec == AVC and avc contains seq header + if (sound_format == 10 && aac_packet_type == 0) + return 1; + + return 0; +} + +/** + * Checks RTMP packet and return 1 when it contains an AVC header +*/ +static int rtmp_packet_is_avc_video_header(RTMPPacket *pkt) { + uint8_t codec_id; + uint8_t avc_packet_type; + + if ((!pkt) || (pkt->size < 2) || pkt->type != RTMP_PT_VIDEO) + return 0; + + codec_id = pkt->data[0] & 0xF; + avc_packet_type = pkt->data[1]; + // Check codec == AVC and avc contains seq header + if (codec_id == 7 && avc_packet_type == 0) + return 1; + + return 0; +} + +/** + * Checks RTMP packet and return 1 when it contains video IDR point +*/ +static int rtmp_packet_is_video_avc_IDR(RTMPPacket *pkt) { + uint8_t frame_type; + uint8_t codec_id; + + if ((!pkt) || (pkt->size < 1) || pkt->type != RTMP_PT_VIDEO) + return 0; + + frame_type = (pkt->data[0] & 0xF0) >> 4; + codec_id = pkt->data[0] & 0xF; + // Check codec == AVC and videoFrame == Keyframe / seekable (assuming that means IDR) + if (codec_id == 7 && frame_type == 1) + return 1; + + return 0; +} + +/** + * Checks RTMP packet and return 1 when it contains onMetadata info +*/ +static int rtmp_packet_is_onMetadata_packet(RTMPPacket *pkt) { + uint8_t commandbuffer[64]; + int stringlen; + GetByteContext gbc; + + if ((!pkt) || (pkt->size < 10) || pkt->type != RTMP_PT_NOTIFY) + return 0; + + bytestream2_init(&gbc, pkt->data, pkt->size); + if (ff_amf_read_string(&gbc, commandbuffer, sizeof(commandbuffer),&stringlen)) + return 0; + + // onMetadata is prepended by "@setDataFrame" + if (!strcmp(commandbuffer, "@setDataFrame")) + return 1; + + return 0; +} + static int rtmp_write(URLContext *s, const uint8_t *buf, int size) { RTMPContext *rt = s->priv_data; @@ -2960,6 +3108,8 @@ static int rtmp_write(URLContext *s, const uint8_t *buf, int size) const uint8_t *buf_temp = buf; uint8_t c; int ret; + int execute_reconnection = 0; + int is_idr = 0; do { if (rt->skip_bytes) { @@ -2988,8 +3138,13 @@ static int rtmp_write(URLContext *s, const uint8_t *buf, int size) bytestream_get_be24(&header); rt->flv_size = pktsize; - if (pkttype == RTMP_PT_VIDEO) + if (pkttype == RTMP_PT_VIDEO) { channel = RTMP_VIDEO_CHANNEL; + rt->has_video = 1; + } + if (pkttype == RTMP_PT_AUDIO) { + rt->has_audio = 1; + } if (((pkttype == RTMP_PT_VIDEO || pkttype == RTMP_PT_AUDIO) && ts == 0) || pkttype == RTMP_PT_NOTIFY) { @@ -3047,7 +3202,155 @@ static int rtmp_write(URLContext *s, const uint8_t *buf, int size) } } - if ((ret = rtmp_send_packet(rt, &rt->out_pkt, 0)) < 0) + // Check if a reconnection is required + // Per interval + if ((rt->reconnect_interval > 0) && + (rt->out_pkt.timestamp >= (rt->last_reconnect_timestamp + rt->reconnect_interval * 1000))) { + rt->last_reconnect_timestamp = rt->out_pkt.timestamp; + rt->force_reconnection_now = 1; + av_log(s, AV_LOG_TRACE, + "trigered internal interval reconnection\n"); + } + // Per go away signal + if (rt->go_away_received > 0) { + rt->go_away_received = 0; + rt->force_reconnection_now = 1; + av_log(s, AV_LOG_TRACE, + "detected go away signal from the peer\n"); + } + + if (rtmp_packet_is_avc_video_header(&rt->out_pkt)) { + // Save last video header + if (rt->last_avc_seq_header_pkt.size) { + av_log(s, AV_LOG_DEBUG, + "freeing last video header packet saved\n"); + ff_rtmp_packet_destroy(&rt->last_avc_seq_header_pkt); + } + // Save AVC seq header packet + if ((ret = ff_rtmp_packet_clone(&rt->last_avc_seq_header_pkt, &rt->out_pkt)) < 0) { + return ret; + } + av_log(s, AV_LOG_DEBUG, "saved video header packet\n"); + } else if (rtmp_packet_is_aac_audio_header(&rt->out_pkt)) { + // Save last audio header + if (rt->last_aac_seq_header_pkt.size) { + av_log(s, AV_LOG_DEBUG, + "freeing last audio header packet saved\n"); + ff_rtmp_packet_destroy(&rt->last_aac_seq_header_pkt); + } + // Save AAC seq header packet + if ((ret = ff_rtmp_packet_clone(&rt->last_aac_seq_header_pkt, &rt->out_pkt)) < 0) { + return ret; + } + av_log(s, AV_LOG_DEBUG, "saved audio header packet\n"); + } else if (rtmp_packet_is_onMetadata_packet(&rt->out_pkt)) { + // Save last onMetadata packet + if (rt->last_metadata_pkt.size) { + av_log(s, AV_LOG_DEBUG, + "freeing last onMetadata packet saved\n"); + ff_rtmp_packet_destroy(&rt->last_metadata_pkt); + } + // Save onMetadata packet + if ((ret = ff_rtmp_packet_clone(&rt->last_metadata_pkt, &rt->out_pkt)) < 0) { + return ret; + } + av_log(s, AV_LOG_DEBUG, "saved onMetadata packet\n"); + } + + // Reconnection has been requested + if (rt->force_reconnection_now >= 1) { + // Check if packet is video IDR + is_idr = rtmp_packet_is_video_avc_IDR(&rt->out_pkt); + av_log(s, AV_LOG_DEBUG, + "looking for the right disconnect point. Is IDR: %d, " + "has_video: %d, has_audio: %d, state: %d, " + "last_avc_seq_header_pkt.size: %d, " + "last_aac_seq_header_pkt.size: %d\n", + is_idr, rt->has_video, rt->has_audio, rt->state, + rt->last_avc_seq_header_pkt.size, + rt->last_aac_seq_header_pkt.size); + + if (rt->has_video && rt->has_audio && + (rt->state == STATE_PUBLISHING)) { + // If we only video let's do the reconnection in an IDR + // frame when we have both headers saved + if (is_idr && rt->last_avc_seq_header_pkt.size && + rt->last_aac_seq_header_pkt.size) + execute_reconnection = 1; + } else if (rt->has_video && !rt->has_audio && + (rt->state == STATE_PUBLISHING)) { + // If we have video and NO audio let's do the reconnection + // in an IDR frame when we have video header saved + if (is_idr && rt->last_avc_seq_header_pkt.size) + execute_reconnection = 1; + } else if (!rt->has_video && + rt->has_audio & (rt->state == STATE_PUBLISHING)) { + // If we have only audio let's do the reconnection when we + // have the audio header saved + if (rt->last_aac_seq_header_pkt.size) + execute_reconnection = 1; + } else { + av_log(s, AV_LOG_DEBUG, + "reconnection is requested but can NOT be executed " + "now, waiting! rt->state: %d, has_video: %d, " + "has_audio: %d, is_idr: %d\n", + rt->state, rt->has_video, rt->has_audio, is_idr); + } + } + + if (execute_reconnection) { + execute_reconnection = 0; + + av_log(s, AV_LOG_DEBUG, + "executing reconnection. rt->flv_off: %d, rt->flv_size: " + "%d\n", + rt->flv_off, rt->flv_size); + + if ((ret = rtmp_reconnect(s)) < 0) + return ret; + + // Reconnect executed, clear the flag + rt->force_reconnection_now = 0; + + av_log(s, AV_LOG_DEBUG, + "reconnected. rt->flv_off: %d, rt->flv_size: %d\n", + rt->flv_off, rt->flv_size); + + // Send last video header if it is saved + if (rt->last_avc_seq_header_pkt.size) { + av_log(s, AV_LOG_DEBUG, + "sending last saved video header\n"); + rt->last_avc_seq_header_pkt.timestamp = + rt->out_pkt.timestamp; + if ((ret = rtmp_send_packet( + rt, &rt->last_avc_seq_header_pkt, 0, 0)) < 0) + return ret; + } + + // Send last audio header if it is saved + if (rt->last_aac_seq_header_pkt.size) { + av_log(s, AV_LOG_DEBUG, + "sending last saved audio header\n"); + rt->last_aac_seq_header_pkt.timestamp = + rt->out_pkt.timestamp; + if ((ret = rtmp_send_packet( + rt, &rt->last_aac_seq_header_pkt, 0, 0)) < 0) + return ret; + } + + // Send last onMetadata packet, optional + if (rt->last_metadata_pkt.size) { + av_log(s, AV_LOG_DEBUG, + "sending last saved onMetadata header\n"); + rt->last_metadata_pkt.timestamp = rt->out_pkt.timestamp; + if ((ret = rtmp_send_packet(rt, &rt->last_metadata_pkt, 0, + 0)) < 0) + return ret; + } + } + + // Send actual packet + if ((ret = rtmp_send_packet(rt, &rt->out_pkt, 0, 1)) < 0) return ret; rt->flv_size = 0; rt->flv_off = 0; @@ -3118,6 +3421,7 @@ static const AVOption rtmp_options[] = { {"listen", "Listen for incoming rtmp connections", OFFSET(listen), AV_OPT_TYPE_INT, {.i64 = 0}, INT_MIN, INT_MAX, DEC, "rtmp_listen" }, {"tcp_nodelay", "Use TCP_NODELAY to disable Nagle's algorithm", OFFSET(tcp_nodelay), AV_OPT_TYPE_INT, {.i64 = 0}, 0, 1, DEC|ENC}, {"timeout", "Maximum timeout (in seconds) to wait for incoming connections. -1 is infinite. Implies -rtmp_listen 1", OFFSET(listen_timeout), AV_OPT_TYPE_INT, {.i64 = -1}, INT_MIN, INT_MAX, DEC, "rtmp_listen" }, + {"rtmp_reconnect_time", "Interval (in seconds) to force a client reconnection, it is based on media time. By default is 0 (no reconnection)", OFFSET(reconnect_interval), AV_OPT_TYPE_INT, {.i64 = 0}, 0, INT_MAX, ENC }, { NULL }, };