yuanbiao
/
emqx
1
0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #10752 from thalesmg/fix-pulsar-redact-auth-r50 

fix(pulsar): redact error reason
This commit is contained in:
Thales Macedo Garitezi 2023-05-19 16:47:05 -03:00 committed by GitHub
parent 9aaa0b6fe1 4327e40740
commit bb4fef3cfd
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
apps/emqx_bridge_pulsar/src/emqx_bridge_pulsar_impl_producer.erl
View File

@ -100,7 +100,7 @@ on_start(InstanceId, Config) ->
msg => "failed_to_start_pulsar_client", msg => "failed_to_start_pulsar_client",
instance_id => InstanceId, instance_id => InstanceId,
pulsar_hosts => Servers, pulsar_hosts => Servers,
reason => Reason reason => emqx_utils:redact(Reason, fun is_sensitive_key/1)
}), }),
throw(failed_to_start_pulsar_client) throw(failed_to_start_pulsar_client)
end, end,
@ -332,7 +332,7 @@ start_producer(Config, InstanceId, ClientId, ClientOpts) ->
#{ #{
instance_id => InstanceId, instance_id => InstanceId,
kind => Kind, kind => Kind,
reason => Error, reason => emqx_utils:redact(Error, fun is_sensitive_key/1),
stacktrace => Stacktrace stacktrace => Stacktrace
} }
), ),
@ -419,3 +419,6 @@ get_producer_status(Producers) ->
partition_strategy(key_dispatch) -> first_key_dispatch; partition_strategy(key_dispatch) -> first_key_dispatch;
partition_strategy(Strategy) -> Strategy. partition_strategy(Strategy) -> Strategy.
is_sensitive_key(auth_data) -> true;
is_sensitive_key(_) -> false.