Merge pull request #8517 from zhongwencool/fix-bad-swagger-format

fix: fix bad swagger format

CHANGES-5.0.md

@@ -13,6 +13,7 @@
 * Fix statistics related system topic name error
 * Fix AuthN JWKS SSL schema. Using schema in `emqx_schema`. [#8458](https://github.com/emqx/emqx/pull/8458)
 * `sentinel` field should be required when AuthN/AuthZ Redis using sentinel mode. [#8458](https://github.com/emqx/emqx/pull/8458)
+* Fix bad swagger format. [#8517](https://github.com/emqx/emqx/pull/8517)
 
 ## Enhancements
 

apps/emqx_authn/src/emqx_authn_user_import_api.erl

@@ -66,15 +66,7 @@ schema("/authentication/:id/import_users") ->
             tags => ?API_TAGS_GLOBAL,
             description => ?DESC(authentication_id_import_users_post),
             parameters => [emqx_authn_api:param_auth_id()],
-            'requestBody' => #{
-                content => #{
-                    'multipart/form-data' => #{
-                        schema => #{
-                            filename => file
-                        }
-                    }
-                }
-            },
+            'requestBody' => emqx_dashboard_swagger:file_schema(filename),
             responses => #{
                 204 => <<"Users imported">>,
                 400 => error_codes([?BAD_REQUEST], <<"Bad Request">>),
@@ -89,15 +81,7 @@ schema("/listeners/:listener_id/authentication/:id/import_users") ->
             tags => ?API_TAGS_SINGLE,
             description => ?DESC(listeners_listener_id_authentication_id_import_users_post),
             parameters => [emqx_authn_api:param_listener_id(), emqx_authn_api:param_auth_id()],
-            'requestBody' => #{
-                content => #{
-                    'multipart/form-data' => #{
-                        schema => #{
-                            filename => file
-                        }
-                    }
-                }
-            },
+            'requestBody' => emqx_dashboard_swagger:file_schema(filename),
             responses => #{
                 204 => <<"Users imported">>,
                 400 => error_codes([?BAD_REQUEST], <<"Bad Request">>),

apps/emqx_authz/src/emqx_authz_api_sources.erl

@@ -564,6 +564,10 @@ authz_sources_type_refs() ->
 bin(Term) -> erlang:iolist_to_binary(io_lib:format("~p", [Term])).
 
 status_metrics_example() ->
+    #{
+        'metrics_example' => #{
+            summary => <<"Showing a typical metrics example">>,
+            value =>
                 #{
                     resource_metrics => #{
                         matched => 0,
@@ -617,6 +621,8 @@ status_metrics_example() ->
                             status => connected
                         }
                     ]
+                }
+        }
     }.
 
 create_authz_file(Body) ->

apps/emqx_dashboard/src/emqx_dashboard_api.erl

@@ -180,7 +180,6 @@ field(username_in_path) ->
     {username,
         mk(binary(), #{
             desc => ?DESC(username),
-            'maxLength' => 100,
             example => <<"admin">>,
             in => path,
             required => true

apps/emqx_dashboard/src/emqx_dashboard_swagger.erl

@@ -24,6 +24,7 @@
 -export([namespace/0, namespace/1, fields/1]).
 -export([schema_with_example/2, schema_with_examples/2]).
 -export([error_codes/1, error_codes/2]).
+-export([file_schema/1]).
 
 -export([filter_check_request/2, filter_check_request_and_translate_body/2]).
 
@@ -165,6 +166,20 @@ error_codes(Codes = [_ | _], MsgDesc) ->
             })}
     ].
 
+file_schema(FileName) ->
+    #{
+        content => #{
+            'multipart/form-data' => #{
+                schema => #{
+                    type => object,
+                    properties => #{
+                        FileName => #{type => string, format => binary}
+                    }
+                }
+            }
+        }
+    }.
+
 %%------------------------------------------------------------------------------
 %% Private functions
 %%------------------------------------------------------------------------------

apps/emqx_gateway/src/emqx_gateway.app.src

@@ -1,7 +1,7 @@
 %% -*- mode: erlang -*-
 {application, emqx_gateway, [
     {description, "The Gateway management application"},
-    {vsn, "0.1.1"},
+    {vsn, "0.1.2"},
     {registered, []},
     {mod, {emqx_gateway_app, []}},
     {applications, [kernel, stdlib, grpc, emqx, emqx_authn]},

apps/emqx_gateway/src/emqx_gateway_api_authn_user_import.erl

@@ -124,15 +124,7 @@ schema("/gateway/:name/authentication/import_users") ->
             #{
                 desc => ?DESC(emqx_gateway_api_authn, import_users),
                 parameters => params_gateway_name_in_path(),
-                'requestBody' => #{
-                    content => #{
-                        'multipart/form-data' => #{
-                            schema => #{
-                                filename => file
-                            }
-                        }
-                    }
-                },
+                'requestBody' => emqx_dashboard_swagger:file_schema(filename),
                 responses =>
                     ?STANDARD_RESP(#{204 => <<"Imported">>})
             }
@@ -145,15 +137,7 @@ schema("/gateway/:name/listeners/:id/authentication/import_users") ->
                 desc => ?DESC(emqx_gateway_api_listeners, import_users),
                 parameters => params_gateway_name_in_path() ++
                     params_listener_id_in_path(),
-                'requestBody' => #{
-                    content => #{
-                        'multipart/form-data' => #{
-                            schema => #{
-                                filename => file
-                            }
-                        }
-                    }
-                },
+                'requestBody' => emqx_dashboard_swagger:file_schema(filename),
                 responses =>
                     ?STANDARD_RESP(#{204 => <<"Imported">>})
             }