Skip to content
This repository has been archived by the owner on Aug 13, 2018. It is now read-only.

Some messages not picked up #33

Closed
eliihen opened this issue Nov 20, 2015 · 9 comments
Closed

Some messages not picked up #33

eliihen opened this issue Nov 20, 2015 · 9 comments
Assignees

Comments

@eliihen
Copy link
Member

eliihen commented Nov 20, 2015

Hi Jan. Nice work with this extension, keep up the good work!

It seems websocket-monitor has issues with some messages. My company produces many pages that stream market data, and the first thing the client does after getting upgraded to WS is send a subscribe string with the data that component would like to receive. In my tests, this client does not pick up these subscribe strings at all.

Steps to reproduce

In Chrome, the strings show in green as subscribe?.... In websocket-monotor, they do not show, only the later response from server to this subscribe string shows up.

WS-monitor 0.4.1 Firefox 45.0a1 (2015-11-20) Fedora Linux

@janodvarko janodvarko self-assigned this Nov 20, 2015
@janodvarko
Copy link
Member

Thanks for the report, I'll investigate it.

Jan (aka Honza)

@janodvarko
Copy link
Member

@bakulf: I've been trying the STR and it really looks like some WS messages are missing?
Could this be a problem with the platform API?

Honza

@tuukka
Copy link
Contributor

tuukka commented Mar 29, 2016

I'm trying to debug MQTT over WebSockets but only some initial handshake is visible: one binary frame sent, one binary frame received, although there's a constant flow of messages. Here's my test page: http://dev.hsl.fi/mqtt/browser/

janodvarko added a commit that referenced this issue Mar 29, 2016
@janodvarko
Copy link
Member

@tuukka:

Here is a test build with patch included, does it help?
http://softwareishard.com/temp/websocketmonitor/websocketmonitor-0.5.3issue33.xpi

Honza

@tuukka
Copy link
Contributor

tuukka commented Mar 29, 2016

@janodvarko I don't see any change (Firefox 45.0 on Ubuntu) :-(

@janodvarko
Copy link
Member

@tuukka: ah, sorry, I forgot to include the patch in the build, here is a new version:
http://softwareishard.com/temp/websocketmonitor/websocketmonitor-0.5.3issue33-2.xpi

Honza

@tuukka
Copy link
Contributor

tuukka commented Mar 29, 2016

@janodvarko Thanks, that did the trick!

@tuukka
Copy link
Contributor

tuukka commented Mar 30, 2016

I also compared the output to the websocket frame view in Chromium, and they match.

@janodvarko
Copy link
Member

Thanks for the verification!
Honza

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants