Date objects are not handled properly #35

Closed
squamos opened this Issue Oct 27, 2012 · 1 comment

2 participants

@squamos
> var msgpack = require('msgpack');
undefined
> var obj = {d: new Date()}
undefined
> msgpack.unpack(msgpack.pack(obj));
{ d: {} }
> var msgpack = require('msgpack');
undefined
> var obj = {d: new Date()}
undefined
> obj
{ d: Fri Oct 26 2012 18:32:26 GMT-0700 (PDT) }
> msgpack.unpack(msgpack.pack(obj));
{ d: {} }

This is a big problem for us. A quick fix is to toString Date objects when packing in msgpack.cc? Other thoughts?

@godsflaw
Collaborator

I merged the pull request a while back, now we have a new issue with this not compiling on windows.

#46

@godsflaw godsflaw closed this Jun 14, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment