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

Atmosphere Jersey api and WebRTC not work well together #905

Closed
alfonsodomenici opened this issue Feb 14, 2013 · 8 comments
Closed

Atmosphere Jersey api and WebRTC not work well together #905

alfonsodomenici opened this issue Feb 14, 2013 · 8 comments

Comments

@alfonsodomenici
Copy link

I have provided two samples of using Atmosphere with Webrtc to establish an audio-video communication between two browser.
The first sample using Atmosphere Handler api works well.
The second sample using Atmosphere Jersey api don't works.

@carloslizarazo
Copy link

Hi Alfonso

Where can i find these samples ?
I'm looking for a way to use Atmosphere with RTC ... and it's pretty difficult to find some light in this subject

Thanks for your attention !!!!!!! ...

@alfonsodomenici
Copy link
Author

Hi Carlos,

Please send me an email and I'll send you the working sample.

The sample with Atmosphere Jersey API for now does not work.
I hope that Jeanfrancois will able to find time to solve the issue..
I have not been able to solve it.

Alfonso

2013/2/21 carloslizarazo notifications@github.com

Hi Alfonso

Where can i find these samples ?
I'm looking for a way to use Atmosphere with RTC ... and it's pretty
difficult to find some light in this subject

Thanks for your attention !!!!!!! ...


Reply to this email directly or view it on GitHubhttps://github.com//issues/905#issuecomment-13871233.

@joelbars
Copy link

Hi Mr. Alfonso,

It's been a long time since this issue, I have the same question of Carlos, where can I find the samples. I want to see if I can help with the one that doesn't work too (if it was not fixed yet).

Thanks in advance.

@veeramarni
Copy link

Mr Alfonso,

Can you please send me the sample as well to my email marni.srikanth@gmail.com? I will look into the issue. Let me know which application server/media server you are using for this.

@jfarcand
Copy link
Member

OK with the following really ugly hack, I can make it work

    @POST
    //@Broadcast(writeEntity = false)
    public void publish(@FormParam("message") String message,
            @PathParam("topic") Broadcaster topic) {
        logger.info("Sending message {} to channel {}", new Object[] {
                message, topic.getID() });
        try {
            Thread.sleep(1000);
        } catch (InterruptedException e) {
            e.printStackTrace();
        }
        //return new Broadcastable(message, "", topic);
        topic.broadcast(message);
    }

So it seems there is an issue inside Jersey or the underlying WebSocket that cause the message to be send out of order, causing the application to fail. I will soon add the sample in the workspace, so @veeramarni you can take a look to see if you spot something. Thanks!

@veeramarni
Copy link

@jfarcand let me know once you have the sample ready.

@jfarcand
Copy link
Member

OK, both samples are here. The AtmosphereHandler works, Jersey is broken unless you add the ugly hack. Any help appreciated. I've tested using 2 machines with FF and Chrome as Opera and Safari aren't supporting web-rtc properly (at least from the script).

@jfarcand
Copy link
Member

jfarcand commented Apr 1, 2014

Won't fix that.

@jfarcand jfarcand closed this as completed Apr 1, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants