From c54527857a076342f3fb45d974e7561d30c4ef8c Mon Sep 17 00:00:00 2001 From: Serge Tupchii Date: Thu, 31 Aug 2023 16:32:19 +0300 Subject: [PATCH] fix(emqx_bridge_sqlserver): use empty password by default as it is not a required field --- apps/emqx_bridge_sqlserver/src/emqx_bridge_sqlserver.app.src | 2 +- .../src/emqx_bridge_sqlserver_connector.erl | 5 ++--- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/apps/emqx_bridge_sqlserver/src/emqx_bridge_sqlserver.app.src b/apps/emqx_bridge_sqlserver/src/emqx_bridge_sqlserver.app.src index 3aa8b3b68..530578d73 100644 --- a/apps/emqx_bridge_sqlserver/src/emqx_bridge_sqlserver.app.src +++ b/apps/emqx_bridge_sqlserver/src/emqx_bridge_sqlserver.app.src @@ -1,6 +1,6 @@ {application, emqx_bridge_sqlserver, [ {description, "EMQX Enterprise SQL Server Bridge"}, - {vsn, "0.1.2"}, + {vsn, "0.1.3"}, {registered, []}, {applications, [kernel, stdlib, emqx_resource, odbc]}, {env, []}, diff --git a/apps/emqx_bridge_sqlserver/src/emqx_bridge_sqlserver_connector.erl b/apps/emqx_bridge_sqlserver/src/emqx_bridge_sqlserver_connector.erl index b6ad15ab9..3e1abbeba 100644 --- a/apps/emqx_bridge_sqlserver/src/emqx_bridge_sqlserver_connector.erl +++ b/apps/emqx_bridge_sqlserver/src/emqx_bridge_sqlserver_connector.erl @@ -173,7 +173,6 @@ on_start( #{ server := Server, username := Username, - password := Password, driver := Driver, database := Database, pool_size := PoolSize, @@ -200,7 +199,7 @@ on_start( Options = [ {server, to_bin(Server)}, {username, Username}, - {password, Password}, + {password, emqx_secret:wrap(maps:get(password, Config, ""))}, {driver, Driver}, {database, Database}, {pool_size, PoolSize} @@ -320,7 +319,7 @@ conn_str([{database, Database} | Opts], Acc) -> conn_str([{username, Username} | Opts], Acc) -> conn_str(Opts, ["UID=" ++ str(Username) | Acc]); conn_str([{password, Password} | Opts], Acc) -> - conn_str(Opts, ["PWD=" ++ str(Password) | Acc]); + conn_str(Opts, ["PWD=" ++ str(emqx_secret:unwrap(Password)) | Acc]); conn_str([{_, _} | Opts], Acc) -> conn_str(Opts, Acc).