434395 - WebSocket / memory leak, WebSocketSession not cleaned up in abnormal closure cases
+ Cleanup for read, bug fix for this occurred in prior commits surrounding CLOSE handling
This commit is contained in:
parent
5db6f65354
commit
f9fad8f49f
|
@ -155,7 +155,7 @@ public class BlockheadServer
|
||||||
public void echoMessage(int expectedFrames, TimeUnit timeoutUnit, int timeoutDuration) throws IOException, TimeoutException
|
public void echoMessage(int expectedFrames, TimeUnit timeoutUnit, int timeoutDuration) throws IOException, TimeoutException
|
||||||
{
|
{
|
||||||
LOG.debug("Echo Frames [expecting {}]",expectedFrames);
|
LOG.debug("Echo Frames [expecting {}]",expectedFrames);
|
||||||
IncomingFramesCapture cap = readFrames(expectedFrames,timeoutUnit,timeoutDuration);
|
IncomingFramesCapture cap = readFrames(expectedFrames,timeoutDuration,timeoutUnit);
|
||||||
// now echo them back.
|
// now echo them back.
|
||||||
for (Frame frame : cap.getFrames())
|
for (Frame frame : cap.getFrames())
|
||||||
{
|
{
|
||||||
|
@ -329,15 +329,6 @@ public class BlockheadServer
|
||||||
return len;
|
return len;
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* @deprecated use {@link #readFrames(int, int, TimeUnit)} for correct parameter order
|
|
||||||
*/
|
|
||||||
@Deprecated
|
|
||||||
public IncomingFramesCapture readFrames(int expectedCount, TimeUnit timeoutUnit, int timeoutDuration) throws IOException, TimeoutException
|
|
||||||
{
|
|
||||||
return readFrames(expectedCount,timeoutDuration,timeoutUnit);
|
|
||||||
}
|
|
||||||
|
|
||||||
public IncomingFramesCapture readFrames(int expectedCount, int timeoutDuration, TimeUnit timeoutUnit) throws IOException, TimeoutException
|
public IncomingFramesCapture readFrames(int expectedCount, int timeoutDuration, TimeUnit timeoutUnit) throws IOException, TimeoutException
|
||||||
{
|
{
|
||||||
LOG.debug("Read: waiting for {} frame(s) from client",expectedCount);
|
LOG.debug("Read: waiting for {} frame(s) from client",expectedCount);
|
||||||
|
|
Loading…
Reference in New Issue