'onmessage' not triggering in node.js about eventsource HOT 3 CLOSED

This library does indeed aim to follow the EventSource API implemented in browsers. There are many tests verifying that it does.

If you believe you've found a bug, your first step should be to prove that there is a bug with a failing test. (You skipped this step).

Your second step should be to fix the failing test.

PR #49 breaks one existing test, and you didn't add a new test.

So as it stands I think your interpretation of the spec is incorrect, but I'd be willing to reconsider if you provide a failing test and provide a link to the section of the spec we're not implementing correctly.

from eventsource.

@m-garg I think this is related to #45, which we rejected. Thoughts?

from eventsource.

Sorry for the misunderstanding. I though the EventSource.onmessage callback should be emitted regardless of the event type sent by the sse. I just tested it on Google Chrome, and the event type sent be the event-stream is being emitted, not the 'message' event. The 'message' event is only emitted only when there is no event type sent by the event-stream or the 'message' event type itself. This is exactly what this library does.
Closing the issue. Thank you for your time.

from eventsource.