turnpike
turnpike copied to clipboard
Don't send nil Arguments or ArgumentsKw
https://github.com/tavendo/WAMP/blob/master/spec/basic.md#call-1 https://github.com/tavendo/WAMP/blob/master/spec/basic.md#empty-arguments-and-keyword-arguments
Specifically in RESULT messages, but we should check INVOCATION and possibly others.
#9 would hopefully prevent things like this from happening in the future, but I don't know if the current test framework actually tests WAMP or if it just tests WebSockets.
If such a thing doesn't exist, perhaps we should create one?
@beatgammit said:
Fixing this in the serializer is probably the best place to do it, since that prevents null from ever accidentally making it in.
We can also add a simple check in a few places like this