Cannot read property '_$onMessage' of undefined #167
Assignees
Labels
Comments
ronag
changed the title
|
Seems to happen during reconnection: |
|
Here is a reproducible example: import deepstream from 'deepstream.io-client-js'
const ds = deepstream('0.0.0.0:6021')
.login({}, (success, authData) => {
if (!success) {
console.log({ authData }, 'Deepstream Authentication Failed. Shutting down...')
setTimeout(() => process.exit(1), 500)
}
})
ds.on('connectionStateChanged', connectionState => {
console.log({ connectionState }, 'Deepstream Connection State Changed.')
})
ds.on('error', (error, event, topic) => {
console.log({ error, event, topic }, 'Deepstream Error.')
})
setInterval(() => {
for (let n = 0; n < 8; ++n) {
updateRecord()
}
}, 500)
let x = 0
function updateRecord () {
const record = ds.record.getRecord('test/1')
record.set({ x })
record.discard()
x++
} |
|
Great, thank you! |
|
Any progress on this issue? It's starting to become a bit more urgent for us... |
|
We got this issue and the other two you just commented on put in for the next release, aiming for friday/monday. |
|
The issue it's fixed in a branch, missing some tests. 'use strict'
const deepstream = require('.')
const Promise = require('bluebird')
const ds = deepstream('0.0.0.0:6021')
.login({}, (success, authData) => {
if (!success) {
console.log({ authData }, 'Deepstream Authentication Failed. Shutting down...')
setTimeout(() => process.exit(1), 500)
}
})
ds.on('connectionStateChanged', connectionState => {
console.log({ connectionState }, 'Deepstream Connection State Changed.')
})
ds.on('error', (error, event, topic) => {
console.log({ error, event, topic }, 'Deepstream Error.')
})
setInterval(Promise.coroutine(function*() {
for (let n = 0; n < 8; ++n) {
yield updateRecord()
}
}), 500)
let x = 0
function updateRecord () {
console.log(x)
return new Promise((resolve, reject) => {
const record = ds.record.getRecord('test/1')
record.set({ x: x++ })
record.on( 'discard', () => {
resolve()
})
record.discard()
})
}If you want to do the same with |
ronag
pushed a commit
to nxtedition/deepstream.io-client-js
that referenced
this issue
Jul 26, 2016
yasserf
added a commit
that referenced
this issue
Jul 28, 2016
yasserf
pushed a commit
that referenced
this issue
Jul 28, 2016
yasserf
added a commit
that referenced
this issue
Jul 28, 2016
Bug/#167 multiple discards of the same record
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The text was updated successfully, but these errors were encountered: