From ed9afe33456ff036999854606a786d00f7f722f7 Mon Sep 17 00:00:00 2001 From: Serge Tupchii Date: Fri, 1 Sep 2023 19:07:23 +0300 Subject: [PATCH] refactor(emqx_connector): use `hocon_schema:override/2` to make pgsql 'username' field required --- apps/emqx_connector/src/emqx_connector_pgsql.erl | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/apps/emqx_connector/src/emqx_connector_pgsql.erl b/apps/emqx_connector/src/emqx_connector_pgsql.erl index 04ba4fd51..5cc25bfa1 100644 --- a/apps/emqx_connector/src/emqx_connector_pgsql.erl +++ b/apps/emqx_connector/src/emqx_connector_pgsql.erl @@ -85,13 +85,10 @@ server() -> adjust_fields(Fields) -> lists:map( fun - ({username, OrigUsernameFn}) -> - {username, fun - (required) -> - true; - (Any) -> - OrigUsernameFn(Any) - end}; + ({username, Sc}) -> + %% to please dialyzer... + Override = #{type => hocon_schema:field_schema(Sc, type), required => true}, + {username, hocon_schema:override(Sc, Override)}; (Field) -> Field end,