4ib3r / StompBrokerJS

NodeJS StompBroker
MIT License
35 stars 28 forks source link

Server messages don't have destination header. #32

Open Caleb-Bolton opened 2 years ago

Caleb-Bolton commented 2 years ago

Server messages sent by a StompBrokerJS server don't have the destination: header described in the STOMP protocol for server messages.

This line https://github.com/4ib3r/StompBrokerJS/blob/04ca0d3587c75f8798a55634ec582c065bdc80ad/stompServer.js#L356 sets the subscription header with the destination value. Instead of setting a subscription header, the destination: header should be populated with the destination value instead to match the STOMP protocol.

Maybe I am mistaken?