red5-client RTMPT unable to connect

[solomax]

Hello All,

I'm trying to connect to red5-server 1.0.3 using red5-client 1.0.3 using RTMPT protocol

connect failed with following log:

18:00:15.898 [Thread-0] DEBUG o.a.h.i.conn.tsccm.ConnPoolByRoute - Notifying no-one, there are no waiting threads

18:00:15.898 [Thread-0] DEBUG o.r.c.net.rtmpt.RTMPTClientConnector - Got an id D1JJ8VAERDJDI

18:00:15.948 [Thread-0] DEBUG o.r.c.net.rtmpt.RTMPTClientConnector - RTMPT handling exception

java.lang.UnsupportedOperationException: Not implemented

at org.red5.client.net.rtmp.RTMPConnManager.createConnection(RTMPConnManager.java:166) ~[red5-client-1.0.3-RELEASE.jar:1.0.3-RELEASE]

at org.red5.client.net.rtmp.RTMPConnManager.createConnection(RTMPConnManager.java:1) ~[red5-client-1.0.3-RELEASE.jar:1.0.3-RELEASE]

at org.red5.client.net.rtmpt.RTMPTClientConnector.openConnection(RTMPTClientConnector.java:172) ~[red5-client-1.0.3-RELEASE.jar:1.0.3-RELEASE]

at org.red5.client.net.rtmpt.RTMPTClientConnector.run(RTMPTClientConnector.java:88) ~[red5-client-1.0.3-RELEASE.jar:1.0.3-RELEASE]

18:00:15.948 [Thread-0] ERROR o.a.o.s.webstart.RTMPTScreenShare - null

According to the log the issue is caused by https://github.com/Red5/red5-client/blob/master/src/main/java/org/red5/client/net/rtmp/RTMPConnManager.java#L166

Can it be fixed workarounded somehow?

currently blocker for us :(((

Thanks a lot in advance!

[Mondain]

Not sure why this is firing, but you could try calling createConnection without the session id.

--

---
You received this message because you are subscribed to the Google Groups "red5" group.
To unsubscribe from this group and stop receiving emails from it, send an email to red5interest...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

--

http://gregoire.org/
https://github.com/Red5

[solomax]

Thanks for the quick answer Paul,

What I'm actually doing is calling:

RTMPTClient.connect("localhost", 80, "openmeetings/2", this);

I'll try to isolate the issue and will send you code example

[solomax]

Just have retested, no additional call are performed

this one line results the error

RTMPTClient.connect("localhost", 80, "openmeetings/2", this);

could you please check this?

[solomax]

Would it be better to create issue on github?

[Mondain]

I consolidated the connection managers in the red5-client; give the latest code a try and let me know if its working properly.

[Mondain]

If the issue persists after the latest mods, yes create an issue.

[solomax]

Created pull-request for this?

Do you have any plans for the releases of red5-client?

[Chris Allen]

That's an excellent question.

Overall we tend to just grab the pieces of Red5 Client that we need to do various tasks on other projects, but I can see it being valuable as a stand-alone tool.

I do think the Red5 Client project warrants its own separate releases. I just don't think we've ever thought of it that way. Let me discuss with the team and see what we can come up with. 

-Chris

[solomax]

Thanks a lot Chris

maybe it can be "point" release (1.0.3.1 for ex.) don't like to push on you but this is key feature of Openmeetings :)

[Mondain]

It may need its own release, but i'm unsure how that will look...

[solomax]

Hello Paul, Chris,

were you able to discuss this? do you have any plans for the release?

I would like to release Openmeetings with this issue fixed :)

Thanks in advance!

[Chris Allen]

So are you saying the bug is now fixed and working for you guys? I don't think that was ever clear to us. 

In terms of an official release we are working on a plan to do that. 

[solomax]

Sorry, most probably this is because of my English :(

I have tested latest revision from github and it works as expected.

The only issue I see is: AFAIU red5-client will need latest red5-server, I'm I right?

I would really appreciate If this can be resolved somehow. Maybe working version can be tagged on github, so I can perform checkout by this tag in our build ....

Thanks in advance!

[Chris Allen]

Well we (Infrared5) have been working on a project for testing the loads of RTMP servers based on Bees with Machine Guns (https://github.com/newsapps/beeswithmachineguns). It spins up AWS instances in order to consume RTMP streams pointed at a server. It's pretty awesome stuff and super useful. We are thinking of calling it Bees with Laser Guns.

We plan on releasing that project as open source, and it leverages the Red5 Client project. So with that, we are motivated to come up with a solution that will help you as well as us. :) Tagging the version in Github might be the easiest first step.

As for Red5 Client needing Red5 Server, they both share dependencies, but that's pulled from red5-core I believe.

-Chris

[solomax]

Great news :)

Actually red5-client seems to have red5-server as dependency

And if I remember correctly new red5-client.jar will not work with red5-server 1.0.3 :(

[Chris Allen]

Ah okay, in that case it sounds like we need to do some clean up on the project structure. I wish we had more time in the day. :)

-Chris

[solomax]

Thanks a lot!

please let me know if I can help somehow

[Mondain]

The client should not depend on the server, I'll verify that and if it does i'll remove the dep. Client should only depend upon the I/O project / jar.

[solomax]

https://github.com/Red5/red5-client/blob/master/pom.xml

contains direct dependency to the server

Almost every class contains imports from server project:

import org.red5.server.api.event.IEvent;

import org.red5.server.api.event.IEventDispatcher;

import org.red5.server.api.service.IPendingServiceCall;

import org.red5.server.api.service.IPendingServiceCallback;

import org.red5.server.net.rtmp.event.IRTMPEvent;

import org.red5.server.net.rtmp.event.Notify;

import org.red5.server.net.rtmp.status.StatusCodes;

import org.red5.server.stream.message.RTMPMessage;

import org.red5.server.net.remoting.RemotingClient;

import org.red5.server.net.remoting.RemotingHeader;

[Mondain]

Ok, fair enough.. However, I think that its overkill to have to include the server bits as a dep for the client. This is something I can and shall resolve.

[solomax]

please let me know if I can help

I would love to

[solomax]

I don't like the idea of pushing you on this, but
Maybe you can tell what is the planned release date for the next release?
Or maybe you can create the tag for current stable version

[Mondain]

We don't have a date for the next release, but there are a couple of things we're trying to address at the moment.

--

---
You received this message because you are subscribed to the Google Groups "red5" group.
To unsubscribe from this group and stop receiving emails from it, send an email to red5interest...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

[solomax]

Hello Paul!

I can extract "red5-server-common" from the client and send you pull-request

or maybe there are better ways to help :)

[Mondain]

I'd be interested to see what that would look like, especially since I haven't had time to do it myself. Go for it.

[solomax]

done:

https://github.com/solomax/red5-server-common

https://github.com/solomax/red5-server

https://github.com/solomax/red5-client

I would remove RTMPTServlet from server-common but it is impossible without code re-writing

If you like it I can clean-up dependencies (add excludes)

and check how everything works :)

Client tests are commented since they will require additional classes to be moved to server-common

[Mondain]

Awesome! and man that was fast! I'll take a look at this today.

[Mondain]

I created the https://github.com/Red5/red5-server-common project, but I haven't gone through all your changes yet. What is the best way to apply your work, after I'm finished?

[solomax]

I can perform additional testing and prepare pull-request in case you prefer it this way

Of you can merge my forked version with yours :)

I'm not that good in git to propose the best way :(

Please let me know if I can help somehow :)

[Mondain]

I can just do a diff based on the local repos and just update the main projects; I'm not that great with git either. I put the server-common snapshot in the red5 maven repo, so that should help in testing. I've done some cursory testing with the server and it seems to work fine. If you want to test, I would appreciate the help since I've got to get to work now.

[solomax]

OK, I'll double-check everything tomorrow, clean-up dependencies and will send you pull request :)

[solomax]

I walk through the dependencies and it seems like I see no libraries I can clean up :)

Can you please merge these changes so we can see how good it will be? :)

[Mondain]

Yes, I can do it, but I may not get to it today since I had to reinstall Ubuntu on my dev box this weekend thanks to a video driver and uefi issue.

[Mondain]

Ok all the class moves should be completed; everything built properly here and I updated the server-common maven artifacts to reflect a couple mods.

[solomax]

Thanks a lot!

Any plans for the release date? :)

[Mondain]

Before 2015 and after I figure out the connection local issue.

[solomax]

Hello Paul,

it seems like connection local issue is resolved, any other blockers? :)

[Mondain]

No blockers that I'm aware of.. We're on-track for a release today time permitting.

[solomax]

Great news!

Maybe we also will be able to release until 2015 :)