NuKVM/xen-orchestra
3
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix(xo-server/collection/redis#{add,update}): cast to string before inserting in db

Browse Source 
Fixes https://xcp-ng.org/forum/post/51933
Fixes #6359

Introduced by 36b94f745
This commit is contained in:
Julien Fontanet 2022-08-07 13:27:25 +02:00
parent c5d2726faa
commit d27b6bd49d
1 changed files with 10 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
packages/xo-server/src/collection/redis.mjs
View File

@ -8,7 +8,6 @@ import ignoreErrors from 'promise-toolbox/ignoreErrors'
import isEmpty from 'lodash/isEmpty.js' import isEmpty from 'lodash/isEmpty.js'
import map from 'lodash/map.js' import map from 'lodash/map.js'
import omit from 'lodash/omit.js' import omit from 'lodash/omit.js'
import pickBy from 'lodash/pickBy.js'
import { v4 as generateUuid } from 'uuid' import { v4 as generateUuid } from 'uuid'
import Collection, { ModelAlreadyExists } from '../collection.mjs' import Collection, { ModelAlreadyExists } from '../collection.mjs'
@ -156,13 +155,16 @@ export default class Redis extends Collection {
} }
const key = `${prefix}:${id}` const key = `${prefix}:${id}`
const promises = [ const props = {}
redis.del(key), for (const name of Object.keys(model)) {
redis.hSet( if (name !== 'id') {
key, const value = model[name]
pickBy(model, (value, name) => value !== undefined && name !== 'id') if (value !== undefined) {
), props[name] = String(value)
] }
}
}
const promises = [redis.del(key), redis.hSet(key, props)]
// Update indexes. // Update indexes.
forEach(indexes, index => { forEach(indexes, index => {