Subscribing to non-existing topic causes parser error

friends-of-reactphp / stomp

STOMP bindings for ReactPHP.

MIT License

115 stars 42 forks source link

Subscribing to non-existing topic causes parser error #28

Closed mrferos closed 11 years ago

mrferos commented 11 years ago

Specifically in line 36 of \React\Stomp\Protocol\Parser.php

<?php
/*** snip ***/
public function parseFrameData($data)
{
        $frame = new Frame();

        list($head, $body) = explode("\n\n", $data, 2);

I don't know the protocol well enough to implement a proper fix for compensating when $data is empty.

mrferos commented 11 years ago

Forgot to mention, this isn't really something that interrupts my own work as I don't dynamically subscribe to topics or anything. Just noticed it when I was setting up from the example and forgot to change what I was subscribing to.

Cheers, Andres.

mrferos commented 11 years ago

Sorry gentleman, I just realized it was my own error. I configured the port to be the JMS port.