Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TypeError: Cannot set property '3' of undefined #92

Open
mndmx31 opened this issue Sep 30, 2015 · 16 comments
Open

TypeError: Cannot set property '3' of undefined #92

mndmx31 opened this issue Sep 30, 2015 · 16 comments

Comments

@mndmx31
Copy link

mndmx31 commented Sep 30, 2015

TypeError: Cannot set property '3' of undefined
at collectStats [as _onTimeout] lib/api.js:43:25
at Timer.listOnTimeout [as ontimeout] timers.js:110:15

2015-09-30 09:55:45 [master] API died, spawning replacement...

@eightsixeight
Copy link

same here

@sammy007
Copy link
Collaborator

sammy007 commented Oct 6, 2015

Same here, I am using old copy of node_modules for now. This is a problem in one of dependencies.

@eightsixeight
Copy link

You have a copy ?

From: Sammy Libre [mailto:[email protected]]
Sent: Tuesday, October 06, 2015 12:27 PM
To: zone117x/node-cryptonote-pool [email protected]
Cc: Frank [email protected]
Subject: Re: [node-cryptonote-pool] TypeError: Cannot set property '3' of undefined (#92)

Same here, I am using old copy of node_modules for now. This is a problem in one of dependencies.


Reply to this email directly or view it on GitHubhttps://github.com//issues/92#issuecomment-145919005.

@sammy007
Copy link
Collaborator

sammy007 commented Oct 6, 2015

Try upgrade node-redis to 2.1.0 there is a bug fixed with argument mutation when using redis.multi. This should fix problem. Please report a result. Just run npm upgrade.

@sammy007
Copy link
Collaborator

sammy007 commented Oct 6, 2015

If it does not work, please downgrade version and detect working one. Please report result here.

@sammy007
Copy link
Collaborator

sammy007 commented Oct 6, 2015

Yeah, hardcode in package.json

"redis": "0.12.1"

@sammy007
Copy link
Collaborator

sammy007 commented Oct 6, 2015

Quick fix in master. Pull changes and run npm update.

@sammy007 sammy007 closed this as completed Oct 6, 2015
@eightsixeight
Copy link

This starts, but now all my payments and fees are NAN in stats.. weird,

From: Sammy Libre [mailto:[email protected]]
Sent: Tuesday, October 06, 2015 12:55 PM
To: zone117x/node-cryptonote-pool [email protected]
Cc: Frank [email protected]
Subject: Re: [node-cryptonote-pool] TypeError: Cannot set property '3' of undefined (#92)

Quick fix in master. Pull changes and run npm upgrade.


Reply to this email directly or view it on GitHubhttps://github.com//issues/92#issuecomment-145926366.

@sammy007
Copy link
Collaborator

sammy007 commented Oct 6, 2015

What is your setup? I am using slightly modified version of this pool and this fix works for me like a charm.

@sammy007 sammy007 reopened this Oct 6, 2015
@sammy007
Copy link
Collaborator

sammy007 commented Oct 6, 2015

node 0.10.40, node-redis 0.12.1. Delete node_modules and run npm update again. It must work.

@sammy007
Copy link
Collaborator

sammy007 commented Oct 6, 2015

Also, i am running redis-server 2.8.4.

@eightsixeight
Copy link

Yes it works BUT, like I said before the fix, it messed my REDIS I think all amount and fee = nan in payments tab… any idea ?

From: Sammy Libre [mailto:[email protected]]
Sent: Tuesday, October 06, 2015 1:27 PM
To: zone117x/node-cryptonote-pool [email protected]
Cc: Frank [email protected]
Subject: Re: [node-cryptonote-pool] TypeError: Cannot set property '3' of undefined (#92)

Also, i am running redis-server 2.8.4.


Reply to this email directly or view it on GitHubhttps://github.com//issues/92#issuecomment-145935099.

@sammy007
Copy link
Collaborator

sammy007 commented Oct 6, 2015

Is it fresh setup? If yes, then it's maybe ok, I can't remember actually how it goes on fresh setup without any data in redis.

@eightsixeight
Copy link

No its:

Had issues with payments not going out..
So I regitted the source in a new dir
Copied my config from old dir to new dir

Ran.. it fucked redis..

Re got the soure

Applied the redis 0.12.1

Reran..

All shows as NAN

From: Sammy Libre [mailto:[email protected]]
Sent: Tuesday, October 06, 2015 5:34 PM
To: zone117x/node-cryptonote-pool [email protected]
Cc: Frank [email protected]
Subject: Re: [node-cryptonote-pool] TypeError: Cannot set property '3' of undefined (#92)

Is it fresh setup? If yes, then it's maybe ok, I can't remember actually how it goes on fresh setup without any data in redis.


Reply to this email directly or view it on GitHubhttps://github.com//issues/92#issuecomment-146008396.

@sammy007
Copy link
Collaborator

sammy007 commented Oct 6, 2015

No idea. I fall into this issue by upgrading server and running npm update. I noticed bug from the title, so I downgraded redis and it works for me now. No idea, try to add additional hiredis parser right before redis. I am not sure I am right and even not talking bullshit, cuz I have no time to play with different setups, etc at this time. Just some guesses. Redis downgrade can't fuck your data, I am using it on my production http://hashinvest.net.

@eightsixeight
Copy link

Nah something else did… also the admin not working no more… this is a clean install even tried on a clean db.

From: Sammy Libre [mailto:[email protected]]
Sent: Tuesday, October 06, 2015 5:57 PM
To: zone117x/node-cryptonote-pool [email protected]
Cc: Frank [email protected]
Subject: Re: [node-cryptonote-pool] TypeError: Cannot set property '3' of undefined (#92)

No idea. I fall into this issue by upgrading server and running npm update. I noticed bug from the title, so I downgraded redis and it works for me now. No idea, try to add additional hiredis parser right before redis. I am not sure I am right and even not talking bullshit, cuz I have no time to play with different setups, etc at this time. Just some guesses. Redis downgrade can't fuck your data, I am using it on my production hashinvest.net.


Reply to this email directly or view it on GitHubhttps://github.com//issues/92#issuecomment-146013497.

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

No branches or pull requests

3 participants