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

完善 MySQL 账号管理,支持新增、授权、改密和删除 #452

Merged
merged 2 commits into from
Sep 20, 2019

Conversation

hhyo
Copy link
Owner

@hhyo hhyo commented Sep 17, 2019

相关issue:#23

  • 支持 MySQL 账号创建、授权、改密和删除,暂不支持取消授权和ALL权限授权操作
  • 使用django-mirage-field对用户名和密码数据进行加密
  • 优化一个会话管理的风险点,不执行前端传来的SQL,由后端自行构建
  • 账号名和密码校验规则待完善
  • 测试用例待完善

image

image

image

@codecov
Copy link

codecov bot commented Sep 17, 2019

Codecov Report

Merging #452 into master will decrease coverage by 1.35%.
The diff coverage is 16.44%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #452      +/-   ##
==========================================
- Coverage    84.6%   83.24%   -1.36%     
==========================================
  Files          71       72       +1     
  Lines        9192     9383     +191     
==========================================
+ Hits         7777     7811      +34     
- Misses       1415     1572     +157
Impacted Files Coverage Δ
sql/db_diagnostic.py 11.42% <0%> (-0.58%) ⬇️
sql/instance.py 46.19% <0%> (+3.39%) ⬆️
sql/models.py 95.12% <100%> (+0.13%) ⬆️
sql/urls.py 100% <100%> (ø) ⬆️
sql/views.py 46.19% <50%> (ø) ⬆️
sql/instance_account.py 7.65% <7.65%> (ø)
sql/admin.py 98.26% <87.5%> (-0.81%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1ff6a77...d0c3c2a. Read the comment docs.

@hhyo hhyo force-pushed the feature/manage-mysql-user branch 2 times, most recently from 45b64b5 to d63ab80 Compare September 19, 2019 14:43
@hhyo hhyo marked this pull request as ready for review September 19, 2019 14:43
@hhyo hhyo added this to ALL in 需求汇总 via automation Sep 19, 2019
@hhyo hhyo added the enhancement New feature or request label Sep 19, 2019
@hhyo hhyo merged commit 692dc67 into master Sep 20, 2019
需求汇总 automation moved this from ALL to Done Sep 20, 2019
@hhyo hhyo deleted the feature/manage-mysql-user branch November 10, 2019 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
需求汇总
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

1 participant