Branch: refs/heads/master Home: https://github.com/qos-ch/logback Commit: c9291b7965429eb86450df28119c85479c3fb2ae https://github.com/qos-ch/logback/commit/c9291b7965429eb86450df28119c85479c3... Author: Richard Fearn <richardfearn@gmail.com> Date: 2013-10-19 (Sat, 19 Oct 2013) Changed paths: M logback-classic/src/main/java/ch/qos/logback/classic/net/SocketNode.java Log Message: ----------- LOGBACK-350 Prevent SocketNode constructor blocking due to lack of data from client The constructor creates an ObjectInputStream, and this blocks until the stream header is received from the client. If the client doesn't send a stream header, construction of the SocketNode will hang, preventing SimpleSocketServer from accepting new clients. This change moves the construction of the ObjectInputStream into SocketNode.run. As SimpleSocketServer runs this method in a new thread, a rogue client won't prevent subsequent ones being able to connect. Commit: ef839a099ba2c21748d54db049a3cf9ac710590b https://github.com/qos-ch/logback/commit/ef839a099ba2c21748d54db049a3cf9ac71... Author: Richard Fearn <richardfearn@gmail.com> Date: 2013-10-20 (Sun, 20 Oct 2013) Changed paths: M logback-classic/src/main/java/ch/qos/logback/classic/net/SocketNode.java Log Message: ----------- Mark SocketNode as closed if ObjectInputStream can't be created This prevents a NullPointerException from being thrown, and allows the socketNodeClosing method to be called for the SocketNode. Commit: 7d4c75c2908c2ba604f4bcdf2ba82b7d23b71462 https://github.com/qos-ch/logback/commit/7d4c75c2908c2ba604f4bcdf2ba82b7d23b... Author: Tony Trinh <tony19@gmail.com> Date: 2013-10-25 (Fri, 25 Oct 2013) Changed paths: M logback-classic/src/main/java/ch/qos/logback/classic/net/SocketNode.java Log Message: ----------- Merge pull request #155 from richardfearn/LOGBACK-350 LOGBACK-350 Prevent SocketNode constructor blocking due to lack of data from client Compare: https://github.com/qos-ch/logback/compare/772f7bf5838b...7d4c75c2908c