Przeglądaj źródła

refactor: set authn config at 'low' prio instead of 'hidden'

Zaiming (Stone) Shi 2 lat temu
rodzic
commit
a30e08d06a
1 zmienionych plików z 10 dodań i 4 usunięć
  1. 10 4
      apps/emqx/src/emqx_schema.erl

+ 10 - 4
apps/emqx/src/emqx_schema.erl

@@ -2762,10 +2762,16 @@ str(S) when is_list(S) ->
     S.
     S.
 
 
 authentication(Which) ->
 authentication(Which) ->
-    Desc =
+    {Importance, Desc} =
         case Which of
         case Which of
-            global -> ?DESC(global_authentication);
-            listener -> ?DESC(listener_authentication)
+            global ->
+                %% For root level authentication, it is recommended to configure
+                %% from the dashboard or API.
+                %% Hence it's considered a low-importance when it comes to
+                %% configuration importance.
+                {?IMPORTANCE_LOW, ?DESC(global_authentication)};
+            listener ->
+                {?IMPORTANCE_HIDDEN, ?DESC(listener_authentication)}
         end,
         end,
     %% poor man's dependency injection
     %% poor man's dependency injection
     %% this is due to the fact that authn is implemented outside of 'emqx' app.
     %% this is due to the fact that authn is implemented outside of 'emqx' app.
@@ -2781,7 +2787,7 @@ authentication(Which) ->
     hoconsc:mk(Type, #{
     hoconsc:mk(Type, #{
         desc => Desc,
         desc => Desc,
         converter => fun ensure_array/2,
         converter => fun ensure_array/2,
-        importance => ?IMPORTANCE_HIDDEN
+        importance => Importance
     }).
     }).
 
 
 %% the older version schema allows individual element (instead of a chain) in config
 %% the older version schema allows individual element (instead of a chain) in config