search

tonistiigi / mega

THIS REPO IS NOT MAINTAINED. Unofficial Node.js SDK for Mega
http://tonistiigi.github.com/mega
118 stars 33 forks source link

Bugfix split keys in File Class #3

Closed fabiux82 closed 11 years ago

fabiux82 commented 11 years ago

I have tried using the library with my mega account but I had a bug in the init step invoking the mega function (with autoload option to true). I solved the problem changing in file.js:File the param to crypto.formatKey.

This is the error output: assert.js:102 throw new assert.AssertionError({ ^ AssertionError: Trying to read beyond buffer length at readInt32 (buffer.js:777:12) at Buffer.readInt32BE (buffer.js:795:10) at AES.decryptKey (/Users/fabiux/Documents/mega_example_js/node_modules/mega/lib/crypto/index.js:109:16) at new File (/Users/fabiux/Documents/mega_example_js/node_modules/mega/lib/file.js:25:19) at Storage._importFile (/Users/fabiux/Documents/mega_example_js/node_modules/mega/lib/storage.js:222:32) at Array.forEach (native) at Storage.reload.api.on.deleted (/Users/fabiux/Documents/mega_example_js/node_modules/mega/lib/storage.js:168:16) at Request.API.request as _callback at Request.init.self.callback (/Users/fabiux/Documents/mega_example_js/node_modules/mega/node_modules/request/main.js:122:22) at Request.EventEmitter.emit (events.js:96:17) at Request. (/Users/fabiux/Documents/mega_example_js/node_modules/mega/node_modules/request/main.js:661:16)

tonistiigi commented 11 years ago

Thanks. Was it a shared file that caused it? Haven't seen it myself that "k" contains more than 2 parts.

fabiux82 commented 11 years ago

Yes, I have tried removing a shared directory and there were no problem. When I added the shared folder I had again the problem because the "k" contains 3 parts. Thanks for your wonderful library! I am really enjoying it ;)