-
-
Notifications
You must be signed in to change notification settings - Fork 754
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
Sending a pipe ('|') with streaming causes a javascript exception #1479
Comments
I can't know what character will not be included in a message in e.g. a chat or if pushing some binary data. As said, this is a regression in 2.1 |
@Artur- I don't think this is a regression...this hasn't changed between released version. You may have added a workaround in your code (this is what I suspect). I recommend you use something like You may want to use the b64 interceptor, but performance wise it is slower. |
The problem is 0770b52 which prevents addition of any length information to the message when it contains a | or whatever character is used. So it's actually a regression in atmosphere-runtime. |
@Artur- Ha I see. But the client may fail IMO because of the multiple char. Let me debug it more closely. |
Good catch, this is clearly a regression. Fixed by now. |
Note that this issue is only with Atmosphere 2.1.0, not 2.0.x |
See http://artur.app.fi/vaadin-uitest-7.2.0.dev-526aa193437e667124a652a8a7c870fe9a0ec501-178/run/BarInUIDL?debug&restartApplication.
Tested using jquery javascript 2.1.4. Works at least in 2.0.3
The text was updated successfully, but these errors were encountered: