yuanbiao
/
emqx
1
0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix(shared_sub): Fix crash on dispatch with subscribers disconnected

This commit is contained in:
ieQu1 2022-07-07 10:46:03 +02:00
parent aaaa65786a
commit e145fdbef3
2 changed files with 12 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/emqx.appup.src
View File

@ -1,9 +1,12 @@
%% -*- mode: erlang -*-
%% Unless you know what you are doing, DO NOT edit manually!!
{VSN,
[{"4.3.17",[{load_module,emqx_access_control,brutal_purge,soft_purge,[]}]},
[{"4.3.17",
[{load_module,emqx_shared_sub,brutal_purge,soft_purge,[]},
{load_module,emqx_access_control,brutal_purge,soft_purge,[]}]},
{"4.3.16",
[{load_module,emqx_access_control,brutal_purge,soft_purge,[]},
[{load_module,emqx_shared_sub,brutal_purge,soft_purge,[]},
{load_module,emqx_access_control,brutal_purge,soft_purge,[]},
{load_module,emqx_plugins,brutal_purge,soft_purge,[]},
{load_module,emqx_metrics,brutal_purge,soft_purge,[]},
{load_module,emqx_app,brutal_purge,soft_purge,[]},
@ -652,9 +655,12 @@
{load_module,emqx_message,brutal_purge,soft_purge,[]},
{load_module,emqx_limiter,brutal_purge,soft_purge,[]}]},
{<<".*">>,[]}],
[{"4.3.17",[{load_module,emqx_access_control,brutal_purge,soft_purge,[]}]},
[{"4.3.17",
[{load_module,emqx_shared_sub,brutal_purge,soft_purge,[]},
{load_module,emqx_access_control,brutal_purge,soft_purge,[]}]},
{"4.3.16",
[{load_module,emqx_access_control,brutal_purge,soft_purge,[]},
[{load_module,emqx_shared_sub,brutal_purge,soft_purge,[]},
{load_module,emqx_access_control,brutal_purge,soft_purge,[]},
{load_module,emqx_plugins,brutal_purge,soft_purge,[]},
{load_module,emqx_metrics,brutal_purge,soft_purge,[]},
{load_module,emqx_app,brutal_purge,soft_purge,[]},

4
src/emqx_shared_sub.erl
View File

@ -242,7 +242,7 @@ maybe_nack_dropped(Msg) ->
%% For fresh Ref we send a nack and return true, to note that the inflight is full
{Sender, {fresh, _Group, Ref}} -> nack(Sender, Ref, dropped), drop;
%% For retry Ref we can't reject a message if inflight is full, so we mark it as
%% acknowledged and put it into mqueue
{_Sender, {retry, _Group, _Ref}} -> maybe_ack(Msg), store;
@ -311,7 +311,7 @@ do_pick(Strategy, ClientId, SourceTopic, Group, Topic, FailedSubs) ->
false;
[] ->
%% We try redispatch to subs who dropped the message because inflight was full.
Found = maps_find_by(FailedSubs, fun({SubPid, FailReason}) ->
Found = maps_find_by(FailedSubs, fun(SubPid, FailReason) ->
FailReason == dropped andalso is_alive_sub(SubPid)
end),
case Found of