From patchwork Sat Sep 18 23:35:46 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jordi Cenzano X-Patchwork-Id: 30367 Delivered-To: ffmpegpatchwork2@gmail.com Received: by 2002:a6b:6506:0:0:0:0:0 with SMTP id z6csp255817iob; Sat, 18 Sep 2021 16:36:09 -0700 (PDT) X-Google-Smtp-Source: ABdhPJyH6jlPEliaEiJwTOLonqJ8G2F+Eu9Qi2UR/BkH49pFAY89jAiyCEwGUyBS8E+NXXqjuh4k X-Received: by 2002:a50:d844:: with SMTP id v4mr20702993edj.73.1632008169714; Sat, 18 Sep 2021 16:36:09 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1632008169; cv=none; d=google.com; s=arc-20160816; b=tkauguuziTaAtPrbVryRaw7OjJkGivO8CpzyA+W/7LgDdlIkIrjRO29CShw6ez18Ig b3I2YVCzIfAPEkBNv84CX11O0zj3+HzAl2jaQFcL8082oxD00o5ccZn8bTetPL1mQzZl 9NdEyQ8bs7HmQA1aTDDG8LK9LWhxRjh7MbYzG/TmQCC1mywYeaBk4gvXCyjjgU/V34hZ RIweSl2uhkyOeS1HMNxuvOqi+VQ3BwbugRiFmLP8te7+O/B4qiRK+5kjDiExMQiDUOPW /9mV1Dt276vMjZC9iqBjE416wZCyNlOMmk6BcI+P/VIjOy36dRSPRiH41IHgNJla5uta drdQ== 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=usxwpr0sXQIlgyQI7Yh81DVBEeQFf2G02aZ0hMKeiI/o4JBrGt5h+zLmRYGbl7cumb +66k0vgXO7ggKe6bak8Ke6JHIBmS+QZTJC4qFpaAgXKEHLpysBnIHopqseRAWBAGWQc8 IH5eTTS40xz+k/TanU+yLznuuf+ZRj7Ydr32jHgke9cmZKdUIFj/L1dbEKxbfaWqQa/1 U2bWTsSNHKFewbhgnwKD3IVAtcDJLb2jvMyXFcbdu2KMg5W5u7xH3xluc2FMF0tdNVQa d0dZFeQYasy0nUvooaZG4jKE1+jXyaecT3r8/aWI6XClQVOLZfeNQvipYExVDxkxX4nb O1pg== ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@gmail.com header.s=20210112 header.b=nI988gJF; 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 r14si10707659ejc.394.2021.09.18.16.36.08; Sat, 18 Sep 2021 16:36:09 -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=nI988gJF; 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 D2F4068B0A4; Sun, 19 Sep 2021 02:36:04 +0300 (EEST) X-Original-To: ffmpeg-devel@ffmpeg.org Delivered-To: ffmpeg-devel@ffmpeg.org Received: from mail-wm1-f44.google.com (mail-wm1-f44.google.com [209.85.128.44]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 745B5689B7F for ; Sun, 19 Sep 2021 02:35:58 +0300 (EEST) Received: by mail-wm1-f44.google.com with SMTP id 140so10212094wma.0 for ; Sat, 18 Sep 2021 16:35:58 -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=nI988gJFWyNgYiVeHkux4GL1r1GrP7EGnkZq+dI5j8nO1gKXAf1JdS3q0Kf1kJDfKm b8JgbU77AzyAnIzNPMyLbYjSSC9eDnWfm5qmzEAO3Bsdk07Cp8I6Vd1Sn6zJrXo0wpeg O4/fqsWScS06YauwDHrn6f2H9ri9dyObW9cN33CcLGiNiMWzw2woYleyqyDsilpLCL5c 00mPvv1uIRLgc3xtYETFbRVDAG33GQ8urx0+7Kw5WFtozUt8+z9H2H9rFcGrhM79FoZh +rp90lftKX3GhrhFeQEJH8tsygWUgW1yQoPT7+3MyuHs2d42AqPlRtLMS8QGug5p1I5j 1ZLw== 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=NLVn9t5wo4YalvuMmyzPPMlg48WNPSFk3I9rjxrZ080kck0B1H4paQySL4ULMarKUj surClb7qVVZBs8Mqy13jhUwuhCCgGK0UYWAsW/f+W5WmcFHadMPrzSjFQRnZ3FwSbZ7Z /+vLipDSNQEFyCkKwR/lVWKpwLjKi1iWq9l+wbvM5p4h/b8uN1GjZ9BSyRv5aZmImGW+ GxnOMUBMYAhGOApw8W2mtRMG5meyBO3QaWQM6bbQ9wqIZpn61XE52mlhDNxr2H009U6o SLvBOpO4BFOPwJOYo9Kjfr1vkJ493+yDwi5nzD8MVNC4bcGd5c3SYrg2oZV1J6RAuB1E hNmA== X-Gm-Message-State: AOAM5320AUsJ954fZkZc4SH3ghxmC+LySwe6SyWzS9NXzNiOfau0om6B P0lq5jGC2tC7FPCTTvZJUcnvqqbtdP9MPAuPP1M= X-Received: by 2002:a05:600c:1c9e:: with SMTP id k30mr16801309wms.74.1632008157251; Sat, 18 Sep 2021 16:35:57 -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 i187sm3297199wma.0.2021.09.18.16.35.56 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Sat, 18 Sep 2021 16:35:56 -0700 (PDT) From: Jordi Cenzano To: ffmpeg-devel@ffmpeg.org Date: Sat, 18 Sep 2021 23:35:46 +0000 Message-Id: <20210918233546.32551-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: WhFUiQt4KS9z 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 }, };