Skip to content

Commit

Permalink
fix: fixed namespacing bugs
Browse files Browse the repository at this point in the history
removed this.store.namespace setting

Signed-off-by: Jytesh <44925963+Jytesh@users.noreply.github.com>
  • Loading branch information
Jytesh committed Aug 25, 2021
1 parent c73dafb commit f14cd4f
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
3 changes: 0 additions & 3 deletions packages/core/src/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,11 +21,8 @@ class Keyv extends EventEmitter {
key => (this[key] = normalizedOptions[key])
)

this.store.namespace = this.namespace

if (typeof this.store.on === 'function' && emitErrors) {
this.store.on('error', error => {
console.log('error', error)
this.emit('error', error)
})
}
Expand Down
1 change: 1 addition & 0 deletions packages/redis/src/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@ class KeyvRedis extends EventEmitter {
const get = this.redis.mget.bind(this.redis)
async function * iterate (curs, pattern) {
const [cursor, keys] = await scan(curs, 'MATCH', pattern)
if (!keys.length) return
const values = await get(keys)
for (const i in keys) {
if (Object.prototype.hasOwnProperty.call(keys, i)) {
Expand Down

0 comments on commit f14cd4f

Please sign in to comment.