1606626 - Support for STOMP protocol

Status: RESOLVED FIXED

(DevTools :: Netmonitor, enhancement, P3)

Description

[Jan Honza Odvarko [:Honza] (always need-info? me)]

The WebSocket payload preview (in a side panel) should be able to parse STOMP protocol and offer nicely formatted data to the user.

Read more about STOMP: https://stomp.github.io/

Honza

Comment 1

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Anyone interested in fixing this bug can look at bug:
Bug 1566755 - Support for WAMP protocol
... to get some inspiration.

Honza

Comment 2

[Ben D (:rockingskier)]

I've started taking a look at this. It's looking promising so far.

It'll be my first contribution so I'll no doubt have some quesitons and issues.

First up, what is the process for pulling in dependencies? I'd like to use the official stomp-js parser (or rather, one of the transpiled versions). It uses the MIT license so there should be no problems there. So far I've just copied the entire file and left it unchanged (bar auto formatting). I've looked around the codebase and there seem to be other similar cases.

Follwing on from that... is it better to pull in a dependency or to write your own version? A subset of the Frame would be useful. Would you suggest pulling in the whole thing unchanged, pull in just what is needed or write something more streamlined (but maybe less feature full). It has functionality for creating Frames as well as parsing them. For this work we obviously only need to parse them.

Thanks
Ben

Comment 3

[Ben D (:rockingskier)]

Attachment: Bug 1606626 - Implement STOMP WebSocket message parsing. r=honza

Comment 4

[Ben D (:rockingskier)]

I opted to pull in the esm6 Parser and Frame (I inlined Byte into Frame rather than add another file/dependency).

Parsed STOMP message

Comment 5

[Ben D (:rockingskier)]

OK, lots of linting errors, mainly from the copied STOMP code. I'll deal with them all next week.

Comment 6

[Ben D (:rockingskier)]

This is all working and up to date. Ready for review.

However I get The analysis task source-test-doc-upload failed, in Phabricator.

Otherwise it builds and test OK.

Comment 7

[Jan Honza Odvarko [:Honza] (always need-info? me)]

@Ben: this is great addition, thanks for working on this!

Is there any online page I could use to test STOMP protocol?

Honza

Comment 8

[Ben D (:rockingskier)]

Thanks @Jan :)

I couldn't find a live example site so I made a very basic little app.

You're welcome to use it :)

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/49840267c0dd
Implement STOMP WebSocket message parsing. r=Honza,bomsy

Comment 10

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/49840267c0dd

Comment 11

[Ben D (:rockingskier)]

Amazing, thanks for all your help.