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

Problem with generateToken #4

Closed
freekode opened this Issue Jul 25, 2013 · 3 comments

Comments

Projects
None yet
2 participants
@freekode

freekode commented Jul 25, 2013

I have a got a problem.

buffer.js:784
    throw TypeError('value is out of bounds');
          ^
TypeError: value is out of bounds
    at TypeError (<anonymous>)
    at checkInt (buffer.js:784:11)
    at Buffer.writeUInt32BE (buffer.js:848:5)
    at makePacket (C:\Users\freekode\Dropbox\devel\utils\status-server\node_modules\mcquery\mcquery.js:256:7)
    at Query.send (C:\Users\freekode\Dropbox\devel\utils\status-server\node_modules\mcquery\mcquery.js:222:13)
    at Query.basic_stat (C:\Users\freekode\Dropbox\devel\utils\status-server\node_modules\mcquery\mcquery.js:90:10)
    at C:\Users\freekode\Dropbox\devel\utils\status-server\app\node_modules\streamcraft-status\index.js:38:22
    at C:\Users\freekode\Dropbox\devel\utils\status-server\node_modules\mcquery\mcquery.js:76:9
    at deQueue (C:\Users\freekode\Dropbox\devel\utils\status-server\node_modules\mcquery\mcquery.js:188:8)
    at Socket.<anonymous> (C:\Users\freekode\Dropbox\devel\utils\status-server\node_modules\mcquery\mcquery.js:39:5)

I changed in this string maximum count (was 50000, now I put 222) and it's work. If I change maximum count to 223, will be error.

OS Windows7 x32, node.js 0.10.14

@kmpm

This comment has been minimized.

Owner

kmpm commented Jul 28, 2013

What version of the server are you connecting to, so I can try to replicate the problem ?

@freekode

This comment has been minimized.

freekode commented Aug 5, 2013

I've create example module for shows the problem

@kmpm kmpm closed this in a03c175 Aug 13, 2013

kmpm added a commit that referenced this issue Aug 13, 2013

Merge pull request #5 from kmpm/dev
Merge fix for #4 from Dev
@kmpm

This comment has been minimized.

Owner

kmpm commented Aug 13, 2013

This is published in v0.1.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment