Eclipse/jetty.project
1
0
Fork 0
mirror of https://github.com/jetty/jetty.project.git synced 2025-02-24 08:28:54 +00:00
Code Issues Packages Projects Releases Wiki Activity

Hightlighting double-close issue better in logs

Browse Source
This commit is contained in:
Joakim Erdfelt 2017-06-20 09:55:48 -07:00
parent 8d18118688
commit 03afcc8487
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
jetty-websocket/websocket-tests/src/main/java/org/eclipse/jetty/websocket/tests/AbstractTrackingEndpoint.java
View File

@ -112,9 +112,13 @@ public abstract class AbstractTrackingEndpoint<T>
LOG.debug("onWSClose({}, {})", statusCode, reason);
}
CloseInfo close = new CloseInfo(statusCode, reason);
boolean closeTracked = closeInfo.compareAndSet(null, close);
if (closeInfo.compareAndSet(null, close) == false)
{
LOG.warn("onClose should only happen once - Original Close: " + closeInfo.get());
LOG.warn("onClose should only happen once - Extra/Excess Close: " + close);
fail("onClose should only happen once!");
}
this.closeLatch.countDown();
assertTrue("Close only happened once", closeTracked);
}
protected void onWSError(Throwable cause)