Skip to content

[security] Remove sensitive msg from log#14421

Merged
merlimat merged 3 commits into
apache:masterfrom
hezhangjian:remove-sensitive-msg-from-log
Feb 23, 2022
Merged

[security] Remove sensitive msg from log#14421
merlimat merged 3 commits into
apache:masterfrom
hezhangjian:remove-sensitive-msg-from-log

Conversation

@hezhangjian

Copy link
Copy Markdown
Member

Motivation

Currently, we are print ClientConfiguration& ServerConfiguration password field in to log.

Modifications

  • remove the sensitive field from toString
  • fix a typo

@hezhangjian hezhangjian added the doc-not-needed Your PR changes do not impact docs label Feb 23, 2022
@hezhangjian hezhangjian self-assigned this Feb 23, 2022
@hezhangjian hezhangjian force-pushed the remove-sensitive-msg-from-log branch from c5d5284 to d583052 Compare February 23, 2022 03:20
@hezhangjian

Copy link
Copy Markdown
Member Author

/pulsarbot run-failure-checks

@merlimat merlimat added this to the 2.11.0 milestone Feb 23, 2022
@merlimat merlimat merged commit 66b69f7 into apache:master Feb 23, 2022
@hezhangjian hezhangjian deleted the remove-sensitive-msg-from-log branch February 24, 2022 00:18
nicoloboschi pushed a commit to nicoloboschi/pulsar that referenced this pull request Mar 1, 2022
* Remove sensitive msg from log

* Remove sensitive msg from log

* Remove sensitive msg from log
Nicklee007 pushed a commit to Nicklee007/pulsar that referenced this pull request Apr 20, 2022
* Remove sensitive msg from log

* Remove sensitive msg from log

* Remove sensitive msg from log
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants