lingling
/
tiny-rdm
mirror of https://github.com/tiny-craft/tiny-rdm.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fix: cannot set key on some low version redis with "keepttl" 

perf: add soft delete key
This commit is contained in:
tiny-craft 2023-09-26 16:35:28 +08:00
parent 47e9d908e7
commit 50ca632a5b
2 changed files with 24 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
backend/services/connection_service.go
View File

@ -527,6 +527,7 @@ func (c *connectionService) GetKeyValue(connName string, db int, key string) (re
}
// SetKeyValue set value by key
// @param ttl <= 0 means keep current ttl
func (c *connectionService) SetKeyValue(connName string, db int, key, keyType string, value any, ttl int64) (resp types.JSResp) {
rdb, ctx, err := c.getRedisClient(connName, db)
if err != nil {
@ -536,7 +537,9 @@ func (c *connectionService) SetKeyValue(connName string, db int, key, keyType st
var expiration time.Duration
if ttl < 0 {
if expiration, err = rdb.PTTL(ctx, key).Result(); err != nil {
expiration = redis.KeepTTL
}
} else {
expiration = time.Duration(ttl) * time.Second
}
@ -546,7 +549,11 @@ func (c *connectionService) SetKeyValue(connName string, db int, key, keyType st
resp.Msg = "invalid string value"
return
} else {
_, err = rdb.Set(ctx, key, str, expiration).Result()
_, err = rdb.Set(ctx, key, str, 0).Result()
// set expiration lonely, not "keepttl"
if err == nil && expiration > 0 {
rdb.Expire(ctx, key, expiration)
}
}
case "list":
if strs, ok := value.([]any); !ok {

14
frontend/src/stores/connections.js
View File

@ -533,8 +533,8 @@ const useConnectionStore = defineStore('connections', {
})
return
} else {
// key not exists, remove this key
await this.deleteKey(server, db, key)
// its danger to delete "non-exists" key, just remove from tree view
await this.deleteKey(server, db, key, true)
}
}
@ -1327,12 +1327,15 @@ const useConnectionStore = defineStore('connections', {
* @param {string} connName
* @param {number} db
* @param {string} key
* @param {boolean} [soft] do not try to remove from redis if true, just remove from tree data
* @returns {Promise<boolean>}
*/
async deleteKey(connName, db, key) {
async deleteKey(connName, db, key, soft) {
try {
const { data, success, msg } = await DeleteKey(connName, db, key)
if (success) {
if (soft !== true) {
await DeleteKey(connName, db, key)
}
// update tree view data
this._deleteKeyNode(connName, db, key)
this._tidyNode(connName, db, key, true)
@ -1341,7 +1344,6 @@ const useConnectionStore = defineStore('connections', {
const tab = useTabStore()
tab.emptyTab(connName)
return true
}
} finally {
}
return false