Could you create a JIRA issue for this, so we (and other users) can track this issue? A short sample is not needed, because I can see in the code that this can indeed go wrong. I'll make sure this gets fixed in the next version.
Best regards, Emond On Sat, May 17, 2014 at 3:59 PM, Olivier Dutrieux < [email protected]> wrote: > Hello Emond, > > Very thanks to the update but I found a problem when I use long polling > for the transport : > > When the EventBus loop to the list of AtmosphereResource (on post > method) and if the update is too long of each AtmosphereResource, the > list of AtmosphereResource is update and reorder (because atmosphere > remove and registry the AtmosphereResource due to long polling) and then > the loop on EventBus never stop and update is infinitie. > > To solve the probleme I do that on line 366 of EventBus.java : > > for (AtmosphereResource resource : > ImmutableList.copyOf(broadcaster.getAtmosphereResources())) > > rather than > > for (AtmosphereResource resource : broadcaster.getAtmosphereResources()) > > > I can if you want attach I short sample. > > Duto > > > Le 17/05/2014 01:22, Papegaaij [via Apache Wicket] a écrit : > > I replied to your mail already, but it seems my message was lost in > > the mail > > outage. Atmosphere and the JS are upgraded on the wicket-6.x and > > master branches. I've also fixed the filter. > > > > https://issues.apache.org/jira/browse/WICKET-5589 > > > > Best regards, > > Emond > > > > On Monday 05 May 2014 14:08:20 Olivier Dutrieux wrote: > > > Could you please update the wicket-atmosphere with the last release > > 2.1.4 : > > > I know that this version depend of jquery 2.0.3+ but there is a pure > > > javascript version (no jquery depend) : > > > > > > https://github.com/Atmosphere/atmosphere-javascript > > > > > > And I notice that's your version of TrackMessageSizeFilter don't extend > > the > > > org.atmosphere.client.TrackMessageSizeFilter but it's necessary to be > > use > > > everywhere my atmosphere : check line 72 of this JavaScriptProtocol > > file > > for > > > why : > > > > > > > > > https://github.com/Atmosphere/atmosphere/blob/atmosphere-project-2.1.4/modul > > > > > es/cpr/src/main/java/org/atmosphere/interceptor/JavaScriptProtocol.java#L > > 72 > > > > > > > > I can pull a request if you want. > > > > > > Duto > > > > > > ----- > > > Duto > > > -- > > > View this message in context: > > > > > > http://apache-wicket.1842946.n4.nabble.com/Wicket-atmosphere-tp4665687.html > > > Sent from the Users forum mailing list archive at Nabble.com. > > > > > > --------------------------------------------------------------------- > > > To unsubscribe, e-mail: [hidden email] > > </user/SendEmail.jtp?type=node&node=4665902&i=0> > > > For additional commands, e-mail: [hidden email] > > </user/SendEmail.jtp?type=node&node=4665902&i=1> > > > > > > > > ------------------------------------------------------------------------ > > If you reply to this email, your message will be added to the > > discussion below: > > > http://apache-wicket.1842946.n4.nabble.com/Wicket-atmosphere-tp4665687p4665902.html > > > > To unsubscribe from Wicket atmosphere, click here > > < > http://apache-wicket.1842946.n4.nabble.com/template/NamlServlet.jtp?macro=unsubscribe_by_code&node=4665687&code=b2xpdmllci5kdXRyaWV1eEBwYXN0ZXVyLmZyfDQ2NjU2ODd8LTExODI0MjM1MTg= > >. > > NAML > > < > http://apache-wicket.1842946.n4.nabble.com/template/NamlServlet.jtp?macro=macro_viewer&id=instant_html%21nabble%3Aemail.naml&base=nabble.naml.namespaces.BasicNamespace-nabble.view.web.template.NabbleNamespace-nabble.view.web.template.NodeNamespace&breadcrumbs=notify_subscribers%21nabble%3Aemail.naml-instant_emails%21nabble%3Aemail.naml-send_instant_email%21nabble%3Aemail.naml > > > > > > -- > Olivier Dutrieux > Groupe Projets (Tél : 31 62) > > > ----- > Duto > -- > View this message in context: > http://apache-wicket.1842946.n4.nabble.com/Wicket-atmosphere-tp4665687p4665909.html > Sent from the Users forum mailing list archive at Nabble.com. > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > >
