-
-
Notifications
You must be signed in to change notification settings - Fork 756
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
AtmosphereResource Initial Suspended's uuid is null when using streaming + POST #537
Comments
Yes, that value only apply to WebSocket. There is no handshake for HTTP as there is only one AtmosphereResource per connection. |
You can use the HeaderConfig.X_ATMOSPHERE_TRACKING_ID to retrieve the UUID that was created for that client (the AtmosphereResource associated with the connection that was suspended) |
How can I use the X_ATMOSPHERE_TRACKING_ID to retrieve the UUID? is there a util for this? |
I'm not following you - you're saying that the HeaderConfig.X_ATMOSPHERE_TRACKING_ID and the UUID are the same one? |
The UUID of the client is always set using the Server's UUID, and passed via the HeaderConfig.X_ATMOSPHERE_TRACKING_ID . I need to document that, I know :-) |
To make things simple, I will also set that value for all transport. I will rename to org.atmosphere.cpr.AtmosphereResource.suspended the key value. OK? |
OK unified the processing and now using ApplicationConfig.SUSPENDED_ATMOSPHERE_RESOURCE_UUID will always return the first suspended AtmosphereResource's uuid for all transport. |
Are you sure they should be the same? cause they are different. this |
remind me how do I download the latest version, including your commits? :) |
The easiest way is to clone the workspace on github: % git clone https://github.com/Atmosphere/atmosphere.git
% cd atmosphere
% mvn -DskipTests=true or add the following in your ~/.m2/settings.xml <repository>
<id>oss-snapshot</id>
<url>https://oss.sonatype.org/content/repositories/snapshots/</url>
<snapshots>
<enabled>true</enabled>
</snapshots>
<releases>
<enabled>true</enabled>
</releases>
</repository> and use 1.0.0-SNAPSHOT in your project. |
already figured that one - |
http://postimage.org/image/msskg0ed5/ better image quality :) |
Hum...are you setting that value? I don't see that value, but let me denug. |
My bad, fixing it. |
no problem... let me know when I can test it :) |
OK, pull again :-) |
testing |
are you sure you commited it? |
I was under the impression you used the workspace :-) Let me deploy it manually. |
I'm not sure but it looks like something is messed up with the uuid in the latest snapshot. |
Grrr. Thanks for the help. Let me test and write test before pushing crappy code. |
OK try again. |
Can you deploy it manually ? I tried adding it to the pom but it says dependency not found I've added this: <repository>
<id>oss-snapshot</id>
<url>https://oss.sonatype.org/content/repositories/snapshots/</url>
<snapshots>
<enabled>true</enabled>
</snapshots>
<releases>
<enabled>true</enabled>
</releases>
</repository>
<dependency>
<groupId>org.atmosphere</groupId>
<artifactId>atmosphere-runtime</artifactId>
<version>1.0.0-SNAPSHOT</version>
</dependency> |
Uploaded, test case added. Bt always wrap your code here using ''' |
OK - websocket is now working correcty but there's and issue with streaming. Please see image attached. We're almost there :) |
If I use streaming and then do a POST then this:
request.getAttribute("org.atmosphere.cpr.AtmosphereResource.webSocket.handshake") returns null
when it's used with websocket it works fine.
The text was updated successfully, but these errors were encountered: