balderdashy / sails.io.js

Browser SDK for communicating w/ Sails via sockets
https://sailsjs.com/documentation/reference/web-sockets/socket-client
183 stars 118 forks source link

removes data when cloning data into params. #85

Closed luislobo closed 8 years ago

luislobo commented 8 years ago

If params are passed on the first call, once it resends the request both data and params exists. This fix removes data, so that it does not fail the validation

luislobo commented 8 years ago

@mikermcneil your last commit broke the case when you first call sails.io.js with params, and it resends the request. Then, both data and params exist. I've added a delete for the data attribute.

mikermcneil commented 8 years ago

Thanks @luislobo!

luislobo commented 8 years ago

@mikermcneil No problem! You are welcome