Use 'format_items = map' for config sections by default

- Lists should still have 'format_items = none' as default.
- Remove explicit 'format_items = map' for all sections.

For sections not converted yet:
- Convert to maps, if this is simple.
- For non-trivial cases, leave 'format_items = none' for now.

include/mongoose_config_spec.hrl

@@ -6,7 +6,7 @@
                   validate_keys = any :: mongoose_config_validator:validator(),
                   required = [] :: [mongoose_config_parser_toml:toml_key()] | all,
                   validate = any :: mongoose_config_validator:section_validator(),
-                  format_items = none :: mongoose_config_spec:format_items(),
+                  format_items = map :: mongoose_config_spec:format_items(),
                   process :: undefined | mongoose_config_parser_toml:list_processor(),
                   defaults = #{} :: #{mongoose_config_parser_toml:toml_key() =>
                                          mongoose_config_parser_toml:config_part()},

src/admin_extra/service_admin_extra.erl

@@ -60,8 +60,7 @@ config_spec() ->
                                                         validate = {enum, ?SUBMODS}},
                                         validate = unique}
                 },
-       defaults = #{<<"submods">> => ?SUBMODS},
-       format_items = map
+       defaults = #{<<"submods">> => ?SUBMODS}
       }.
 
 mod_name(ModAtom) ->

src/auth/ejabberd_auth_anonymous.erl

@@ -93,8 +93,7 @@ config_spec() ->
                                            validate = {enum, [sasl_anon, login_anon, both]}}
                 },
        defaults = #{<<"allow_multiple_connections">> => false,
-                    <<"protocol">> => sasl_anon},
-       format_items = map
+                    <<"protocol">> => sasl_anon}
       }.
 
 %% @doc Return true if multiple connections have been allowed in the config file

src/auth/ejabberd_auth_dummy.erl

@@ -47,8 +47,7 @@ config_spec() ->
                  <<"variance">> => #option{type = integer,
                                            validate = positive}},
        defaults = #{<<"base_time">> => 50,
-                    <<"variance">> => 450},
-       format_items = map
+                    <<"variance">> => 450}
       }.
 
 authorize(Creds) ->

src/auth/ejabberd_auth_external.erl

@@ -80,8 +80,7 @@ config_spec() ->
                                           validate = non_empty}
                 },
        required = [<<"program">>],
-       defaults = #{<<"instances">> => 1},
-       format_items = map
+       defaults = #{<<"instances">> => 1}
       }.
 
 -spec check_cache_last_options(mongooseim:host_type()) -> 'cache' | 'no_cache'.

src/auth/ejabberd_auth_http.erl

@@ -51,10 +51,7 @@ stop(_HostType) ->
 
 -spec config_spec() -> mongoose_config_spec:config_section().
 config_spec() ->
-    #section{
-       items = #{<<"basic_auth">> => #option{type = string}},
-       format_items = map
-      }.
+    #section{items = #{<<"basic_auth">> => #option{type = string}}}.
 
 -spec supports_sasl_module(mongooseim:host_type(), cyrsasl:sasl_module()) -> boolean().
 supports_sasl_module(_HostType, cyrsasl_plain) -> true;

src/auth/ejabberd_auth_jwt.erl

@@ -70,8 +70,7 @@ config_spec() ->
                  <<"username_key">> => #option{type = atom,
                                                validate = non_empty}
                 },
-       required = all,
-       format_items = map
+       required = all
       }.
 
 jwt_secret_config_spec() ->
@@ -81,6 +80,7 @@ jwt_secret_config_spec() ->
                  <<"env">> => #option{type = string,
                                       validate = non_empty},
                  <<"value">> => #option{type = string}},
+       format_items = none,
        process = fun ?MODULE:process_jwt_secret/1
       }.
 

src/auth/ejabberd_auth_rdbms.erl

@@ -88,8 +88,7 @@ stop(_HostType) ->
 config_spec() ->
     #section{
        items = #{<<"users_number_estimate">> => #option{type = boolean}},
-       defaults = #{<<"users_number_estimate">> => false},
-       format_items = map
+       defaults = #{<<"users_number_estimate">> => false}
       }.
 
 -spec supports_sasl_module(mongooseim:host_type(), cyrsasl:sasl_module()) -> boolean().

src/auth/ejabberd_auth_riak.erl

@@ -55,8 +55,7 @@ config_spec() ->
     #section{
        items = #{<<"bucket_type">> => #option{type = binary,
                                               validate = non_empty}},
-       defaults = #{<<"bucket_type">> => <<"users">>},
-       format_items = map
+       defaults = #{<<"bucket_type">> => <<"users">>}
       }.
 
 -spec supports_sasl_module(mongooseim:host_type(), cyrsasl:sasl_module()) -> boolean().

src/auth/mongoose_gen_auth.erl

@@ -135,8 +135,7 @@ stop(Mod, HostType) ->
 config_spec(Mod) ->
     case is_exported(Mod, config_spec, 0) of
         true -> Mod:config_spec();
-        false -> #section{items = #{},
-                          format_items = map}
+        false -> #section{}
     end.
 
 -spec supports_sasl_module(ejabberd_auth:authmodule(), mongooseim:host_type(),