Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix: mask custom connection string passwords (CLI & env) #2410

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 6, 2025
Merged

fix: mask custom connection string passwords (CLI & env) #2410

merged 1 commit into from
Apr 6, 2025

Conversation

@indoordiary
Copy link
Contributor

Description

优化了对密码脱敏的逻辑，修复mongodb在使用自定义字符连接时（环境变量中使用DB_CONNECTION_STRING），密码明文输出在日志里。
屏幕截图 2025-03-28 165245

Linked Issues

Additional context

@Innei
Copy link
Member

Innei commented Apr 6, 2025

Thank you

@Innei Innei merged commit e0f5116 into mx-space:master Apr 6, 2025
1 check passed
@sysfox
Copy link

sysfox commented Jul 28, 2025

@indoordiary @Innei
这个修改导致使用默认docker-compose.yml启动时出现如下输出


1Panel-localmx-space-db-Vf9U        | Starting Mix Space
1Panel-localmx-space-db-Vf9U        | ============== Configurations ==============
1Panel-localmx-space-db-Vf9U        | Listen Port: 2333
1Panel-localmx-space-db-Vf9U        | MongoDB: mongodb://mx-mongo:27017/mx-space
1Panel-localmx-space-db-Vf9U        | mongodb://mx-mongo:27017/mx-space
1Panel-localmx-space-db-Vf9U        | Redis: mx-redis:6379
1Panel-localmx-space-db-Vf9U        | Allowed Origins: localhost
1Panel-localmx-space-db-Vf9U        | Config Path: NULL
1Panel-localmx-space-db-Vf9U        | Encryption: false
1Panel-localmx-space-db-Vf9U        | Cluster: false
1Panel-localmx-space-db-Vf9U        | ============================================
1Panel-localmx-space-db-Vf9U        |

MongoDB相关信息出现了两行？

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Innei Innei Innei approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@indoordiary @Innei @sysfox