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

Infinite read loop when server disconnects in PubSub mode #31

Closed
mrunonen opened this issue Jan 22, 2013 · 6 comments
Closed

Infinite read loop when server disconnects in PubSub mode #31

mrunonen opened this issue Jan 22, 2013 · 6 comments

Comments

@mrunonen
Copy link

Connected client that is in PubSub mode seems to go into an infinite loop when the server side disconnects. Looking inside the module, it is this

while (__try_read_sock($sock))

in wait_for_messages() that never returns.

Is this known behaviour or a defect?

@melo
Copy link
Member

melo commented Jan 22, 2013

It's not a known behaviour.

Can you provide:

  • the operating system: we had problems with windows before;
  • which Redis.pm version are you using.

I'll try to reproduce it.

@mrunonen
Copy link
Author

Hi melo,

Thanks, very swift :)

Red Hat Enterprise Linux Server release 5.5 (Tikanga) / Redis.pm 1.958 / perl 5, version 14, subversion 1 (v5.14.1)

@melo
Copy link
Member

melo commented Jan 22, 2013

Ok, pretty sane environment :)

Probably a bug. I'll check it out.

melo added a commit that referenced this issue Jan 22, 2013
… dead

Signed-off-by: Pedro Melo <melo@simplicidade.org>
@melo melo closed this as completed in ecec0fc Jan 22, 2013
@melo
Copy link
Member

melo commented Jan 22, 2013

Please test master if you can... It should be fixed. I'm going to wait for Travis build 49 (build 48 only included the failing test case) to see if everything is kosher and then release 1.959.

@mrunonen
Copy link
Author

Just tried master, works perfectly. Thank you very much & keep up the good work!

@melo
Copy link
Member

melo commented Jan 22, 2013

Cool, glad to know.

Will release 1.959 after lunch, Travis should be done by then...

(@Ovid: this might fix #32, but will not include #33, I need a bit more time to merge #33, master drifted away from you base point; I'll do it tonight).

melo added a commit that referenced this issue Jan 22, 2013
    * __try_read_sock: test error conditions more thoroughly (issue #31)
    * Improve Test::SpawnRedisServer: multi-server support
    * tidyall the code and docs
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