search

simont77 / fakegato-history

Module to emulate Elgato Eve history
MIT License
166 stars 17 forks source link

googleDrive failed callback #74

Closed MaurizioB84 closed 5 years ago

MaurizioB84 commented 5 years ago

when I lunch the script quickstartGoogleDrive.js and retrieve the link when I put the code into command line I receive the following error:

fs.js:128 throw new ERR_INVALID_CALLBACK(); ^

TypeError [ERR_INVALID_CALLBACK]: Callback must be a function at maybeCallback (fs.js:128:9) at Object.writeFile (fs.js:1163:14) at storeToken (/homebridge/node_modules/fakegato-history/lib/googleDrive.js:356:6) at /homebridge/node_modules/fakegato-history/lib/googleDrive.js:337:7 at /homebridge/node_modules/fakegato-history/node_modules/google-auth-library/lib/auth/oauth2client.js:154:5 at Request._callback (/homebridge/node_modules/fakegato-history/node_modules/google-auth-library/lib/transporters.js:106:7) at Request.self.callback (/homebridge/node_modules/fakegato-history/node_modules/request/request.js:185:22) at Request.emit (events.js:198:13) at Request. (/homebridge/node_modules/fakegato-history/node_modules/request/request.js:1161:10) at Request.emit (events.js:198:13)

NorthernMan54 commented 5 years ago

I applied a patch a few minutes ago, it looks like the NodeJS FS Api has changed since this was written.

Please try again

MaurizioB84 commented 5 years ago

Hello,

I tried but same error:

Enter the code from that page here: xxxxxxxxxxxxxxx fs.js:128 throw new ERR_INVALID_CALLBACK(); ^

TypeError [ERR_INVALID_CALLBACK]: Callback must be a function at maybeCallback (fs.js:128:9) at Object.writeFile (fs.js:1163:14) at storeToken (/homebridge/node_modules/fakegato-history/lib/googleDrive.js:356:6) at /homebridge/node_modules/fakegato-history/lib/googleDrive.js:337:7 at /homebridge/node_modules/fakegato-history/node_modules/google-auth-library/lib/auth/oauth2client.js:154:5 at Request._callback (/homebridge/node_modules/fakegato-history/node_modules/google-auth-library/lib/transporters.js:106:7) at Request.self.callback (/homebridge/node_modules/fakegato-history/node_modules/request/request.js:185:22) at Request.emit (events.js:198:13) at Request. (/homebridge/node_modules/fakegato-history/node_modules/request/request.js:1161:10) at Request.emit (events.js:198:13)

NorthernMan54 commented 5 years ago

@simont77 Could you publish the latest package on NPM, the fix is good but I think the user is not getting the latest version.

MaurizioB84 commented 5 years ago

@simont77 Could you publish the latest package on NPM, the fix is good but I think the user is not getting the latest version.

I download the package from npm, maybe is out of sync?

NorthernMan54 commented 5 years ago

Yes it would be, as I can only update GitHub

On Aug 20, 2019, at 12:34 PM, MaurizioB84 notifications@github.com wrote:

@simont77 Could you publish the latest package on NPM, the fix is good but I think the user is not getting the latest version.

I download the package from npm, maybe is out of sync?

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub, or mute the thread.

simont77 commented 5 years ago

Sorry, I’m away from my PC for the next three weeks. I will publish it when I will come back

MaurizioB84 commented 5 years ago

I can download and install direct from git? Can you please suggest the right command? I'm on docker on raspberry

NorthernMan54 commented 5 years ago

Try 

npm install https://github.com/simont77/fakegato-history
MaurizioB84 commented 5 years ago

ok now works. thanks! can you please help me in order to configure in the right way?

from my understanding I need to add into file fakegato-history.js the following rows: var FakeGatoHistoryService = require('fakegato-history')(homebridge); Accessory.log = this.log; this.loggingService = new FakeGatoHistoryService(accessoryType, Accessory, { size:length, // optional - if you still need to specify the length storage:'googleDrive', folder:'fakegatoFolder', // folder on Google drive to persist data, 'fakegato' if empty keyPath:'/place/to/store/my/keys/' // where to find client_secret.json, if empty it will be used the -U homebridge option if present or .homebridge });

is it right? or where I need to add or which command it's needed to be exec?

many thanks for your help

NorthernMan54 commented 5 years ago

How is this for a working sample?

https://github.com/NorthernMan54/homebridge-dht/blob/master/index.js

MaurizioB84 commented 5 years ago

but i need to create a specific js?

simont77 commented 5 years ago

published as 0.5.4

simont77 commented 5 years ago

@NorthernMan54 my installation reports a vulnerability in googleapis that is fixed in version s>39.1.0. We are using version 18.0.0. Could you check if we can update the dependency? Thanks

NorthernMan54 commented 5 years ago

Okay, give me about 48 hours, out of town right now

On Sep 7, 2019, at 2:40 PM, simont77 notifications@github.com wrote:

@NorthernMan54 my installation reports a vulnerability in googleapis that is fixed in version s>39.1.0. We are using version 18.0.0. Could you check if we can update the dependency? Thanks

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

NorthernMan54 commented 5 years ago

@simont77 Needed a couple of minor updates to support the change to latest version, but this is resolved.

simont77 commented 5 years ago

Thanks. Published as v0.5.5