search

ssbc / ssb-blobs

blob gossiping ssb-subprotocol
MIT License
12 stars 11 forks source link

fixed test, and "unwrapped" blobs.add in inject.js #19

Closed austinfrey closed 5 years ago

austinfrey commented 5 years ago

i think this fixes #18 but would appreciate a review. not sure what else that wrap function was meant to do.

arj03 commented 5 years ago

Right, this works because multiblob checks for undefined, not null.

The first message coming through is the sync: true message. This is why the test doesn't error. This should have been filtered with pull.filter((msg) => !msg.sync), instead of pull(2).

This should fix #17 as well.

Nice find! Merged into 1.2.2 :)

austinfrey commented 5 years ago

thanks @arj03!