Events emitted from server not heard by client

[mwilc0x]

Hi, first thank you for this project.

I am working on simple demo to stream some data to the client. I am running into an issue where I am not hearing events which are emitted from server on the client. Here is what my use case looks like:

Server:

server.On("connection", func(so socketio.Socket) {
   log.Println("user connected")

   go tweets()

   so.On("get-tweets-data", func() {
       log.Println("I heard event from client!")
       so.Emit("tweets-data")
   })
   so.On("disconnection", func() {
       log.Println("user disconnected")
   })
})

Client:

socket.on('connect', function(data) {

  socket.emit('get-tweets-data');

  socket.on('tweets-data', function(data) {
    console.log('received data from the server');
  });

});

I heard the initial "get-tweets-data" call from the client, but after that when I try to callback to the client from server something gets lost in translation.

If I move the Emit outside of the On event, it works fine.

server.On("connection", func(so socketio.Socket) {
   log.Println("user connected")

   go tweets()

   so.On("get-tweets-data", func() {
       log.Println("I heard event from client!")
   })
   so.On("disconnection", func() {
       log.Println("user disconnected")
   })
   so.Emit("tweets-data")
})

[mountkin]

Hi, @mjw56
Please try the patch I posted at #53 and see if it works.
By the way, can you print out the return value of so.Emit call and see if it's something like "upgrading"?

[mwilc0x]

Hey @mountkin, I made the change as you had suggested.

I'm seeing that the use case I described above is now working. I'm also seeing that when I print out return value for so.Emit I do see "upgrading" still.

[mountkin]

@mjw56 did you remove the precompiled archives under $GOPATH/pkg/xxx/github.com/googollee/ directory after you apply the patch?

Edit:
I misunderstood your last comment and thought it's still not working. Now if it works please ignore this comment.

[mwilc0x]

@mountkin No worries, it is working now :)

Thank you for your help. Any chance this can make it into master so others don't face this problem?

[googollee]

As I merged the go-engine.io, please check this issue again. Thanks.

[mattmassicotte]

I know its been a while since this issue has been touched, but as of master on April 15th, 2015, I'm experiencing a similar issue. It appears that Emits that occur in the On("connection",..) function work, but all other emits return the "upgrading" error.

[alecthomas]

I'm having the same issue, though not reliably. Sometimes it errors with "upgrading", sometimes it doesn't.

[syhlion]

I'm having the same issue +1

[moul]

Same problem here 👍

[bieleckim]

+1

[pgamaster]

+1 upgrading error under 2000+ clients

[erkie]

Closing this issue because it is old. There are a couple of newer issues with the same problem so please refer to those. Thanks!