delano / redis-dump

Redis to JSON and nack again
https://rubygems.org/gems/redis-dump
MIT License
818 stars 123 forks source link

Issues with serialized data and non-UTF8 encoding #7

Closed shilov closed 11 years ago

shilov commented 12 years ago

I store hashes in Redis by serializing them with the help of MessagePack.

The serialized objects typically look like like some variation of this: \x93\x01\x02\x03

The problem is it completely breaks the usability of redis-dump because Yajl throws a parse error when I try to re-load the file:

ERROR (Yajl::ParseError): lexical error: invalid bytes in UTF8 string.
          421,"type":"zset","value":[["??vid?13289295403089412158?tid?
                     (right here) ------^

Any suggestions on how to approach this issue?

delano commented 12 years ago

Hmm, no, I haven't been able to reproduce it. Thanks for the report.

I'll happy accept a pull request.

CodeMonkeyKevin commented 12 years ago

This is a yajl-ruby gem issue fyi.

delano commented 11 years ago

Just pushed v0.3.4 with this fix. Thanks to meekmichael.