Skip to content

Ensure if a content length is specified, we wait for the full body. - #54

Merged
jmglsn merged 2 commits into
stomp-php:masterfrom
andrewbelcher:master
Jun 18, 2016
Merged

Ensure if a content length is specified, we wait for the full body.#54
jmglsn merged 2 commits into
stomp-php:masterfrom
andrewbelcher:master

Conversation

@andrewbelcher

Copy link
Copy Markdown
Contributor

Fixes #53.

@staabm

staabm commented Jun 14, 2016

Copy link
Copy Markdown
Member

please make sure to include a unit test, which fails without your changes

@jmglsn

jmglsn commented Jun 14, 2016

Copy link
Copy Markdown
Member

@staabm I'll check if we can create an easy backport for version 3.

@staabm

staabm commented Jun 14, 2016

Copy link
Copy Markdown
Member

Awesome, thx guys.

@jmglsn
jmglsn merged commit fd9f079 into stomp-php:master Jun 18, 2016
@jmglsn

jmglsn commented Jun 18, 2016

Copy link
Copy Markdown
Member

Will be included on our next 4.x release.
Null-Byte Handling in version 3.x is working as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants