Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature][Config File Encryption And Decryption] The decrypted result does not need to be printed #6980

Closed
2 of 3 tasks
mayufeng2595 opened this issue Jun 13, 2024 · 3 comments

Comments

@mayufeng2595
Copy link

mayufeng2595 commented Jun 13, 2024

Search before asking

  • I had searched in the feature and found no similar feature requirement.

Description

When my configuration file needs to be decrypted, the decrypted file is printed directly at the info level. This puts the password at risk of being compromised。If you want the configuration file to be printed, you can set whether it is printed or not through the configuration file.

当我的配置文件需要解密时,解密后的文件会直接以info级别的形式打印出来。这样会导致密码存在泄漏的风险。如果要打印配置文件,可以通过配置文件设置是否打印。

Usage Scenario

During the security check, the plaintext of the password is not allowed in the log file。

安全检查的时候,日志文件中不允许出现密码的明文。

Related issues

Can the default encryption field be adjusted to be configurable? Auth and username are currently encrypted. But there are times when you don't want to encrypt these two fields.

默认加密字段是否可以调整为可配置的?目前会对auth 和 username进行加密。但是有些时候并不想对这两个字段加密。

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@EricJoy2048
Copy link
Member

@chl-wxp

@Hisoka-X
Copy link
Member

Fixed by #6987

@Hisoka-X
Copy link
Member

Can the default encryption field be adjusted to be configurable? Auth and username are currently encrypted. But there are times when you don't want to encrypt these two fields.

Please create another issue to follow it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants