brainlag / JavaNSQClient

Fast Java client for NSQ
MIT License
90 stars 57 forks source link

Creating new connection causes NPE instead of graceful fail. #16

Closed klucar closed 8 years ago

klucar commented 8 years ago

Connection.java, lines 70, 71

final NSQFrame response = commandAndWait(ident);
LogManager.getLogger(this).info("Server identification: " + ((ResponseFrame) response).getMessage());

commandAndWait returns null on an InterruptedException, which causes the log line to throw an NPE on response.getMessage()

I haven't been able to reproduce this error yet, but I saw it while testing.