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

Stats() Command fails if Version isn't set #12

Closed
robbiehudson opened this issue Mar 3, 2010 · 2 comments
Closed

Stats() Command fails if Version isn't set #12

robbiehudson opened this issue Mar 3, 2010 · 2 comments

Comments

@robbiehudson
Copy link

If beanstalkd doesn't have a version number (I just downloaded and compiled 1.4.3 and it doesn't) the stats command fails (because there is version).

I modified the config.h of beanstalkd, recompiled and stats now works fine. Pheanstalk should at least cope parsing a message with no version number.

See stack trace:

Fatal error: Uncaught exception 'Pheanstalk_Exception' with message 'YAML parse error for line: version: ' in /media/psf/pp/library/pheanstalk/classes/Pheanstalk/YamlResponseParser.php:62
Stack trace:
#0 /media/psf/pp/library/pheanstalk/classes/Pheanstalk/Connection.php(157): Pheanstalk_YamlResponseParser->parseResponse('OK 802', '---?current-job...')
#1 /media/psf/pp/library/pheanstalk/classes/Pheanstalk.php(395): Pheanstalk_Connection->dispatchCommand(Object(Pheanstalk_Command_StatsCommand))
#2 /media/psf/pp/library/pheanstalk/classes/Pheanstalk.php(329): Pheanstalk->_dispatch(Object(Pheanstalk_Command_StatsCommand))
#3 /media/psf/pp/library/pheanstalk/test.php(15): Pheanstalk->stats('testtube')
#4 {main}

thrown in /media/psf/pp/library/pheanstalk/classes/Pheanstalk/YamlResponseParser.php on line 62

@pda
Copy link
Collaborator

pda commented Mar 11, 2010

Thanks for the report. I couldn't reproduce the missing version number in beanstalkd-1.4.3 but replicated the problem in tests/Pheanstalk/BugfixTest.php

Fixed in 2791352

@robbiehudson
Copy link
Author

Thanks very much!

This issue was closed.
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

2 participants