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

[ManagedService Annotation] Simultaneous write to AtmosphereResponse #1483

Closed
soldierkam opened this issue Feb 27, 2014 · 2 comments
Closed

Comments

@soldierkam
Copy link

I noticed that in some situations server do not send message size (I use TrackMessageSizeFilter). I looked at AtmosphereResponse and I notice field forceAsyncIOWriter which i think is causing problems. When something is writing to response then other attempts to write will be not handled by AsyncIOWriter (so it will not be intercepted by TrackMessageSizeFilter). Right now i do not have example app but i will try to write smth to the end of week.

@soldierkam
Copy link
Author

I changed 'chat' sample app. It is avaiable at soldierkam/atmosphere-samples@84e45d8
You need to refresh webpage few times to get some errors in browser console (typical race condition).
I am gettng smth like that:
Uncaught Error: message length "{"message":"Lorem ipsum dolor si...}99" is not a number

@jfarcand jfarcand added 2.1.1 and removed RC1 labels Feb 28, 2014
@jfarcand
Copy link
Member

Great finding, thanks!

jfarcand added a commit that referenced this issue Feb 28, 2014
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