Unable to handle binary type 2 #11

Open
paulbjensen opened this Issue Sep 14, 2011 · 3 comments

Comments

Projects
None yet
3 participants
@paulbjensen

StackTrace:

exception error: no case clause matching 2
    in function  bson_binary:get_binary/1

Root of StackTrace:

bson/src/bson_binary.erl:132
@pusewicz

This comment has been minimized.

Show comment
Hide comment
@pusewicz

pusewicz Sep 14, 2011

Looks like this is the old binary type (which should be supported):

subtype ::= "\x00"  Binary / Generic
          | "\x01"  Function
          | "\x02"  Binary (Old)
          | "\x03"  UUID
          | "\x05"  MD5
          | "\x80"  User defined

http://bsonspec.org/#/specification

Looks like this is the old binary type (which should be supported):

subtype ::= "\x00"  Binary / Generic
          | "\x01"  Function
          | "\x02"  Binary (Old)
          | "\x03"  UUID
          | "\x05"  MD5
          | "\x80"  User defined

http://bsonspec.org/#/specification

@TonyGen

This comment has been minimized.

Show comment
Hide comment
@TonyGen

TonyGen Sep 14, 2011

Owner

You must have created that data using an old driver. It's easy to add the case to read (but not write) that type. I will do it in a week or so. Submit a pull-request if you want it sooner.
Cheers,
Tony

Owner

TonyGen commented Sep 14, 2011

You must have created that data using an old driver. It's easy to add the case to read (but not write) that type. I will do it in a week or so. Submit a pull-request if you want it sooner.
Cheers,
Tony

@pusewicz

This comment has been minimized.

Show comment
Hide comment
@pusewicz

pusewicz Sep 14, 2011

Cool, that would be great, as I only need to read the data. I'll see if I can submit a patch.

Sent from my iPhone

On 14 Sep 2011, at 19:10, Tony Hannanreply@reply.github.com wrote:

You must have created that data using an old driver. It's easy to add the case to read (but not write) that type. I will do it in a week or so. Submit a pull-request if you want it sooner.
Cheers,
Tony

Reply to this email directly or view it on GitHub:
#11 (comment)

Cool, that would be great, as I only need to read the data. I'll see if I can submit a patch.

Sent from my iPhone

On 14 Sep 2011, at 19:10, Tony Hannanreply@reply.github.com wrote:

You must have created that data using an old driver. It's easy to add the case to read (but not write) that type. I will do it in a week or so. Submit a pull-request if you want it sooner.
Cheers,
Tony

Reply to this email directly or view it on GitHub:
#11 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment