[TYRUS-198] Implement TyrusHttpUpgradeHandler#destroy() Created: 17/Jun/13 Updated: 26/Jun/13 Resolved: 26/Jun/13
|Remaining Estimate:||Not Specified|
|Time Spent:||Not Specified|
|Original Estimate:||Not Specified|
I guess that's the way to find out client disconnects(HttpUpgradeHandler#destroy() specifies about client disconnects) in servlet container
|Comment by Pavel Bucek [ 17/Jun/13 ]|
|Comment by jitu [ 17/Jun/13 ]|
|Comment by Pavel Bucek [ 18/Jun/13 ]|
well, destroy is sometimes called. Unfortunately, it does not seem to be reliable.
When I close browser tab (connection is interrupted), TyrusUpgradeHandler#onError is called with NPE:
line 140 in TyrusUpgradeHandler is servletInputStream.isReady() call.. seems like a bug in servlet impl? Jitu, do you have any comments/insight?
|Comment by jitu [ 18/Jun/13 ]|
I also think that it is servlet container's issue esp grizzly is detecting and informing about disconnect. Adding Shing Wai to the watcher's list.
|Comment by Pavel Bucek [ 26/Jun/13 ]|
fixed in the trunk (rev 691)