Merge pull request #11845 from kjellwinblad/kjell/shared_con/EMQX-11271

fix: crash when listing non bridge_v1 compatible bridge_v2
2023-10-30 20:29:36 +01:00 · 2023-10-30 20:29:36 +01:00 · 9d7ed31c3a
parent c07cf9051e 8fb426e57f
commit 9d7ed31c3a
1 changed files with 7 additions and 3 deletions
--- a/apps/emqx_bridge/src/emqx_bridge_v2.erl
+++ b/apps/emqx_bridge/src/emqx_bridge_v2.erl
@ -233,9 +233,13 @@ list_with_lookup_fun(LookupFun) ->
                fun(Name, _RawConf, Acc) ->
                    [
                        begin
-                            {ok, BridgeInfo} =
-                                LookupFun(Type, Name),
-                            BridgeInfo
+                            case LookupFun(Type, Name) of
+                                {ok, BridgeInfo} ->
+                                    BridgeInfo;
+                                {error, not_bridge_v1_compatible} = Err ->
+                                    %% Filtered out by the caller
+                                    Err
+                            end
                        end
                        | Acc
                    ]