-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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申请查询权限增加系统库 #2746
base: master
Are you sure you want to change the base?
The head ref may contain hidden characters: "\u67E5\u8BE2\u6743\u9650\u589E\u52A0\u7CFB\u7EDF\u5E93X"
MySQL申请查询权限增加系统库 #2746
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2746 +/- ##
=======================================
Coverage 77.30% 77.30%
=======================================
Files 119 119
Lines 16316 16316
=======================================
Hits 12613 12613
Misses 3703 3703 ☔ View full report in Codecov by Sentry. |
@LeoQuote 这个单看看有没有问题,能否合并? |
这个实现方法有点奇怪, 能否直接在引擎侧放开算了 |
可以,我觉可行。再次确认一下?确认的话,我改一下。 |
information_schema的可以放开,不过在SQL工单上面可能还是要做下过滤 |
问题是前端过滤还是后端过滤合理呢? 我想改 EngineBase类get_all_databases(self): 这个方法定义。 |
需求背景:
(select * from information_schema.
TABLES
limit 10; select * from information_schema.COLUMNS
limit 10;)但执行时会报错。
修改内容:
(后端get_all_databases方法,把系统库给屏蔽了,SQL查询和SQL上线不想显示系统库。
但是权限申请希望显示。
)