Merge pull request #10595 from lafirest/fix/password_redact

fix: add new sensitive keywords to redact checklist
This commit is contained in:
lafirest 2023-05-04 16:46:57 +08:00 committed by GitHub
commit e82623d0fb
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 17 additions and 3 deletions

View File

@ -581,6 +581,15 @@ is_sensitive_key(<<"password">>) -> true;
is_sensitive_key(secret) -> true;
is_sensitive_key("secret") -> true;
is_sensitive_key(<<"secret">>) -> true;
is_sensitive_key(secret_key) -> true;
is_sensitive_key("secret_key") -> true;
is_sensitive_key(<<"secret_key">>) -> true;
is_sensitive_key(security_token) -> true;
is_sensitive_key("security_token") -> true;
is_sensitive_key(<<"security_token">>) -> true;
is_sensitive_key(aws_secret_access_key) -> true;
is_sensitive_key("aws_secret_access_key") -> true;
is_sensitive_key(<<"aws_secret_access_key">>) -> true;
is_sensitive_key(_) -> false.
redact(Term) ->

View File

@ -48,7 +48,11 @@ fields(config) ->
{aws_secret_access_key,
mk(
binary(),
#{required => true, desc => ?DESC("aws_secret_access_key")}
#{
required => true,
desc => ?DESC("aws_secret_access_key"),
sensitive => true
}
)},
{pool_size, fun emqx_connector_schema_lib:pool_size/1},
{auto_reconnect, fun emqx_connector_schema_lib:auto_reconnect/1}

View File

@ -52,9 +52,10 @@ fields(config) ->
{secret_key,
mk(
binary(),
#{default => <<>>, desc => ?DESC("secret_key")}
#{default => <<>>, desc => ?DESC("secret_key"), sensitive => true}
)},
{security_token, mk(binary(), #{default => <<>>, desc => ?DESC(security_token)})},
{security_token,
mk(binary(), #{default => <<>>, desc => ?DESC(security_token), sensitive => true})},
{sync_timeout,
mk(
emqx_schema:duration(),