Skip to content
This repository has been archived by the owner on Apr 25, 2023. It is now read-only.

Logging in with Android app makes heimcontrol crash #85

Closed
harmverbeek opened this issue Jul 28, 2014 · 5 comments
Closed

Logging in with Android app makes heimcontrol crash #85

harmverbeek opened this issue Jul 28, 2014 · 5 comments

Comments

@harmverbeek
Copy link

I downnloaded the heimcontrol android app from the Play Store.
When I use it to log in heimcontrol crashes with the following error message:

/home/pi/heimcontrol.js/node_modules/mongodb/lib/mongodb/connection/base.js:242
throw message;
^
Error: Cannot use a writeConcern without a provided callback
at Collection.update (/home/pi/heimcontrol.js/node_modules/mongodb/lib/mongodb/collection/core.js:641:11)
at /home/pi/heimcontrol.js/routes/index.js:515:15
at Db.collection (/home/pi/heimcontrol.js/node_modules/mongodb/lib/mongodb/db.js:495:44)
at /home/pi/heimcontrol.js/routes/index.js:514:29
at /home/pi/heimcontrol.js/node_modules/mongodb/lib/mongodb/cursor.js:162:16
at commandHandler (/home/pi/heimcontrol.js/node_modules/mongodb/lib/mongodb/cursor.js:706:16)
at /home/pi/heimcontrol.js/node_modules/mongodb/lib/mongodb/db.js:1806:9
at Server.Base._callHandler (/home/pi/heimcontrol.js/node_modules/mongodb/lib/mongodb/connection/base.js:442:41)
at /home/pi/heimcontrol.js/node_modules/mongodb/lib/mongodb/connection/server.js:485:18
at MongoReply.parseBody (/home/pi/heimcontrol.js/node_modules/mongodb/lib/mongodb/responses/mongo_reply.js:68:5)

I have no problem when I log in from a browser.

@tjanson
Copy link
Contributor

tjanson commented Jul 28, 2014

Are you using the very latest version of heimcontrol.js (i.e., do a git pull)? Alternatively, can you post your heimcontrol.js/routes/index.js around line 515?

@harmverbeek
Copy link
Author

Hello,

I simply followed the instructions on page
http://ni-c.github.io/heimcontrol.js/get-started.html

But I used
wget http://nodejs.org/dist/v0.10.26/node-v0.10.26.tar.gz
instead of
wget http://nodejs.org/dist/v0.10.18/node-v0.10.18.tar.gz

The file index.js is attached.

Regards,

Harm

From: tjanson notifications@github.com
Reply-To: "ni-c/heimcontrol.js"
<reply+i-38934053-55e92f6b87b45e196534b1dfd4a3826388c27062-8293480@reply.git
hub.com>
Date: Monday 28 July 2014 23:19
To: "ni-c/heimcontrol.js" heimcontrol.js@noreply.github.com
Cc: HV harm@certeza.nl
Subject: Re: [heimcontrol.js] Logging in with Android app makes heimcontrol
crash (#85)

routes/index.js

@tjanson
Copy link
Contributor

tjanson commented Jul 28, 2014

The attachment doesn't show unfortunately (Github converts your email to comments). The result of git log -1 in the heimcontrol directory would work too.

The thing is: I think this bug was fixed very recently and you’re not running the latest code. A git pull should solve it – if you are running the old version.

@harmverbeek
Copy link
Author

Ok, I will pull the latest code.

Harm

From: tjanson notifications@github.com
Reply-To: "ni-c/heimcontrol.js"
<reply+i-38934053-55e92f6b87b45e196534b1dfd4a3826388c27062-8293480@reply.git
hub.com>
Date: Monday 28 July 2014 23:43
To: "ni-c/heimcontrol.js" heimcontrol.js@noreply.github.com
Cc: HV harm@certeza.nl
Subject: Re: [heimcontrol.js] Logging in with Android app makes heimcontrol
crash (#85)

The attachment doesn't show unfortunately (Github converts your email to
comments). The result of git log -1 in the heimcontrol directory would work
too.

The thing is: I think this bug was fixed very recently and you¹re not
running the latest code. A git pull should solve it ­ if you are running the
old version.


Reply to this email directly or view it on GitHub
#85 (comment) .

@harmverbeek
Copy link
Author

Solved in the latest code

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants