fix: redact influxdb tokens in a few logs
This commit is contained in:
parent
6175076f6f
commit
8836494542
|
@ -255,7 +255,7 @@ create(BridgeType, BridgeName, RawConf) ->
|
|||
brige_action => create,
|
||||
bridge_type => BridgeType,
|
||||
bridge_name => BridgeName,
|
||||
bridge_raw_config => RawConf
|
||||
bridge_raw_config => emqx_misc:redact(RawConf)
|
||||
}),
|
||||
emqx_conf:update(
|
||||
emqx_bridge:config_key_path() ++ [BridgeType, BridgeName],
|
||||
|
|
|
@ -137,7 +137,7 @@ create(Type, Name, Conf, Opts0) ->
|
|||
msg => "create bridge",
|
||||
type => Type,
|
||||
name => Name,
|
||||
config => Conf
|
||||
config => emqx_misc:redact(Conf)
|
||||
}),
|
||||
Opts = override_start_after_created(Conf, Opts0),
|
||||
{ok, _Data} = emqx_resource:create_local(
|
||||
|
@ -172,7 +172,7 @@ update(Type, Name, {OldConf, Conf}, Opts0) ->
|
|||
msg => "update bridge",
|
||||
type => Type,
|
||||
name => Name,
|
||||
config => Conf
|
||||
config => emqx_misc:redact(Conf)
|
||||
}),
|
||||
case recreate(Type, Name, Conf, Opts) of
|
||||
{ok, _} ->
|
||||
|
@ -182,7 +182,7 @@ update(Type, Name, {OldConf, Conf}, Opts0) ->
|
|||
msg => "updating_a_non_existing_bridge",
|
||||
type => Type,
|
||||
name => Name,
|
||||
config => Conf
|
||||
config => emqx_misc:redact(Conf)
|
||||
}),
|
||||
create(Type, Name, Conf, Opts);
|
||||
{error, Reason} ->
|
||||
|
|
|
@ -495,15 +495,15 @@ log_and_alarm(IsSuccess, Res, #{kind := ?APPLY_KIND_INITIATE} = Meta) ->
|
|||
%% because nothing is committed
|
||||
case IsSuccess of
|
||||
true ->
|
||||
?SLOG(debug, Meta#{msg => "cluster_rpc_apply_result", result => Res});
|
||||
?SLOG(debug, Meta#{msg => "cluster_rpc_apply_result", result => emqx_misc:redact(Res)});
|
||||
false ->
|
||||
?SLOG(warning, Meta#{msg => "cluster_rpc_apply_result", result => Res})
|
||||
?SLOG(warning, Meta#{msg => "cluster_rpc_apply_result", result => emqx_misc:redact(Res)})
|
||||
end;
|
||||
log_and_alarm(true, Res, Meta) ->
|
||||
?SLOG(debug, Meta#{msg => "cluster_rpc_apply_ok", result => Res}),
|
||||
?SLOG(debug, Meta#{msg => "cluster_rpc_apply_ok", result => emqx_misc:redact(Res)}),
|
||||
do_alarm(deactivate, Res, Meta);
|
||||
log_and_alarm(false, Res, Meta) ->
|
||||
?SLOG(error, Meta#{msg => "cluster_rpc_apply_failed", result => Res}),
|
||||
?SLOG(error, Meta#{msg => "cluster_rpc_apply_failed", result => emqx_misc:redact(Res)}),
|
||||
do_alarm(activate, Res, Meta).
|
||||
|
||||
do_alarm(Fun, Res, #{tnx_id := Id} = Meta) ->
|
||||
|
|
|
@ -1,6 +1,6 @@
|
|||
{application, emqx_conf, [
|
||||
{description, "EMQX configuration management"},
|
||||
{vsn, "0.1.10"},
|
||||
{vsn, "0.1.11"},
|
||||
{registered, []},
|
||||
{mod, {emqx_conf_app, []}},
|
||||
{applications, [kernel, stdlib]},
|
||||
|
|
|
@ -200,8 +200,8 @@ start_client(InstId, Config) ->
|
|||
?SLOG(info, #{
|
||||
msg => "starting influxdb connector",
|
||||
connector => InstId,
|
||||
config => Config,
|
||||
client_config => ClientConfig
|
||||
config => emqx_misc:redact(Config),
|
||||
client_config => emqx_misc:redact(ClientConfig)
|
||||
}),
|
||||
try
|
||||
do_start_client(InstId, ClientConfig, Config)
|
||||
|
@ -236,8 +236,8 @@ do_start_client(
|
|||
?SLOG(info, #{
|
||||
msg => "starting influxdb connector success",
|
||||
connector => InstId,
|
||||
client => Client,
|
||||
state => State
|
||||
client => redact_auth(Client),
|
||||
state => redact_auth(State)
|
||||
}),
|
||||
{ok, State};
|
||||
false ->
|
||||
|
@ -245,7 +245,7 @@ do_start_client(
|
|||
?SLOG(error, #{
|
||||
msg => "starting influxdb connector failed",
|
||||
connector => InstId,
|
||||
client => Client,
|
||||
client => redact_auth(Client),
|
||||
reason => "client is not alive"
|
||||
}),
|
||||
{error, influxdb_client_not_alive}
|
||||
|
@ -255,7 +255,7 @@ do_start_client(
|
|||
?SLOG(info, #{
|
||||
msg => "restarting influxdb connector, found already started client",
|
||||
connector => InstId,
|
||||
old_client => Client0
|
||||
old_client => redact_auth(Client0)
|
||||
}),
|
||||
_ = influxdb:stop_client(Client0),
|
||||
do_start_client(InstId, ClientConfig, Config);
|
||||
|
@ -338,6 +338,13 @@ password(#{password := Password}) ->
|
|||
password(_) ->
|
||||
[].
|
||||
|
||||
redact_auth(Term) ->
|
||||
emqx_misc:redact(Term, fun is_auth_key/1).
|
||||
|
||||
is_auth_key(<<"Authorization">>) -> true;
|
||||
is_auth_key(<<"authorization">>) -> true;
|
||||
is_auth_key(_) -> false.
|
||||
|
||||
%% -------------------------------------------------------------------------------------------------
|
||||
%% Query
|
||||
do_query(InstId, Client, Points) ->
|
||||
|
|
Loading…
Reference in New Issue