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

新增密码仓库的初步支持 #2912

Merged
merged 5 commits into from
Feb 26, 2025
Merged

新增密码仓库的初步支持 #2912

merged 5 commits into from
Feb 26, 2025

Conversation

LeoQuote
Copy link
Collaborator

@LeoQuote LeoQuote commented Feb 26, 2025

有些云厂商或公司内部设置了密码自动轮换, 保证安全, archery 的密码只能是静态的, 只能通过外挂程序来更新数据库内的密码, 本PR带来一个新的功能, 可以在 engine 初始化时自动去请求数据库的账号名和密码, 达到相对的安全.

同时提供了对应的配置项, 供高级使用者提供自己的密码管理器实现, 其中的实现可以完全自己掌控

  1. 新增 DummyMixin , 作为base
  2. 新增 VaultMixin, 作为范例
  3. 配置项中新增 PASSWORD_MIXIN_PATH, 供需要的用户覆盖内置的密码管理器, 替代为自行实现的KMS, 如各大云厂商提供的KMS.

replace #2212

Copy link

codecov bot commented Feb 26, 2025

Codecov Report

Attention: Patch coverage is 87.93103% with 7 lines in your changes missing coverage. Please review.

Project coverage is 78.43%. Comparing base (ab36d56) to head (348e97a).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
sql/models.py 64.28% 5 Missing ⚠️
sql/plugins/password.py 89.47% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2912      +/-   ##
==========================================
+ Coverage   78.40%   78.43%   +0.02%     
==========================================
  Files         124      125       +1     
  Lines       17628    17682      +54     
==========================================
+ Hits        13822    13869      +47     
- Misses       3806     3813       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@LeoQuote LeoQuote merged commit a38cc95 into hhyo:master Feb 26, 2025
10 checks passed
@LeoQuote LeoQuote deleted the add_vault branch February 26, 2025 09:45
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

Successfully merging this pull request may close these issues.

1 participant