chore: don't send subscribe msg if topic list is empty

2022-02-22 14:42:10 +08:00 · 2022-02-22 14:42:10 +08:00 · cc3a4f0c5f
parent 9b53d36571
commit cc3a4f0c5f
1 changed files with 6 additions and 2 deletions
--- a/apps/emqx_auto_subscribe/src/emqx_auto_subscribe.erl
+++ b/apps/emqx_auto_subscribe/src/emqx_auto_subscribe.erl
@ -60,8 +60,12 @@ test(_) ->
 %% hook

 on_client_connected(ClientInfo, ConnInfo, {TopicHandler, Options}) ->
-    TopicTables = erlang:apply(TopicHandler, handle, [ClientInfo, ConnInfo, Options]),
-    self() ! {subscribe, TopicTables};
+    case erlang:apply(TopicHandler, handle, [ClientInfo, ConnInfo, Options]) of
+        [] -> ok;
+        TopicTables ->
+            _ = self() ! {subscribe, TopicTables},
+            ok
+    end;
 on_client_connected(_, _, _) ->
    ok.