fix(redis): 修复普通用户申请权限后无法查询的问题 - #3250
Conversation
|
你好!感谢你反馈的问题/bug，但是你的描述好像是空的，我们需要你完整的信息，这样才能帮你解决问题 如果不知道怎么写，在新建issue的时候有若干个模板可供选择，祝好！ |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1d03d15478
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| var value = (typeof item === "object" && item !== null) | ||
| ? item.value : item; | ||
| var text = (typeof item === "object" && item !== null) | ||
| ? item.text : item; |
There was a problem hiding this comment.
When the selected instance uses Memcached, /api/v1/sqlquery/resources/ returns each node as a one-element array because MemcachedEngine.get_all_databases() appends [db_name] in sql/engines/memcached.py:69-70. Arrays also satisfy this new object check, so both item.value and item.text become undefined; the permission form consequently displays and submits undefined rather than Node - 0, and approved users cannot connect to the requested node. Restrict this branch to objects that actually provide the Redis value/text fields, or retain the prior handling for arrays.
Useful? React with 👍 / 👎.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #3250 +/- ##
=======================================
Coverage 88.34% 88.34%
=======================================
Files 157 157
Lines 29175 29175
=======================================
Hits 25775 25775
Misses 3400 3400 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
修复普通用户申请权限后无法查询的问题 |
LeoQuote
left a comment
LeoQuote
left a comment
There was a problem hiding this comment.
对于数据格式不兼容的问题, 请更改 redis engine 使其产出符合标准的数据
2 participants
No description provided.