Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

streaming transport disconnection event missing #533

Closed
kindlejiang opened this issue Jul 26, 2012 · 2 comments
Closed

streaming transport disconnection event missing #533

kindlejiang opened this issue Jul 26, 2012 · 2 comments

Comments

@kindlejiang
Copy link

when using streaming transport and the browser closed, the server can not trigger onDisconnect(AtmosphereResourceEvent event) , the atmosphere version is 1.0.0 beta3

@jfarcand
Copy link
Member

EDITED: Which connector are you using (just saw you are using Tomcat 7 in your other issue #532. If you are using the BIO Connector this is expected as BIO doesn't detect closed connection

@jfarcand
Copy link
Member

Closing this issue because with the NIO connector, I can clearly reproduce

java.lang.Exception: Stack trace
    at java.lang.Thread.dumpStack(Thread.java:1273)
    at org.atmosphere.samples.chat.ChatAtmosphereHandler$1.onDisconnect(ChatAtmosphereHandler.java:56)
    at org.atmosphere.cpr.AtmosphereResourceImpl.onDisconnect(AtmosphereResourceImpl.java:805)
    at org.atmosphere.cpr.AtmosphereResourceImpl.notifyListeners(AtmosphereResourceImpl.java:749)
    at org.atmosphere.cpr.AtmosphereResourceImpl.notifyListeners(AtmosphereResourceImpl.java:729)
    at org.atmosphere.cpr.DefaultBroadcaster.executeAsyncWrite(DefaultBroadcaster.java:763)
    at org.atmosphere.cpr.DefaultBroadcaster$3.run(DefaultBroadcaster.java:793)
    at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:441)
    at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:303)
    at java.util.concurrent.FutureTask.run(FutureTask.java:138)
    at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
    at java.lang.Thread.run(Thread.java:680)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants