Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix transportMessage() #6

Merged
merged 6 commits into from Jun 17, 2020
Merged

Fix transportMessage() #6

merged 6 commits into from Jun 17, 2020

Conversation

JoseGoncalves
Copy link
Contributor

@JoseGoncalves JoseGoncalves commented Aug 21, 2019

The decode() function called inside transportMessage() is expecting a string parameter, so we need to assure that a string is send when the event object has no data property.

@jenkinskurento
Copy link
Contributor

@jenkinskurento jenkinskurento commented Aug 21, 2019

Hi there, thanks for your Pull Request!

A Kurento member needs to verify that this patch is reasonable to test. In case it is, they should write a comment with the phrase test this please. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by Kurento members will still work. Regular contributors can be whitelisted to skip this step.

@j1elo j1elo merged commit 5b30695 into Kurento:master Jun 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants