search

Red5 / red5-client

Red5 Client
Apache License 2.0
162 stars 140 forks source link

I found bug of rtmp client... #1

Closed TrueTrAxeX closed 9 years ago

TrueTrAxeX commented 10 years ago

Hello all! Please enter to src/test/java/org.red5.client.ClientTest.java, when i enter more than 107 symbols into play filename. I can't view "play" request in wireshark.

http://i.gyazo.com/6fc2e7862cd98602517ff07d0fc2cd43.png Screenshot.

Please fix this bug!

dominick commented 10 years ago

Even when using a very long name, it is not truncated in Red5. The name I used included 202 characters.

screen shot 2014-04-07 at 1 20 26 pm

dominick commented 10 years ago

Please notice that wireshark did not fail for me. What are you using for your RTMPClient?

screen shot 2014-04-07 at 1 25 15 pm

TrueTrAxeX commented 10 years ago

I opened ClientTest.java on windows 8, may therefore be there was this bug. I will check client on linux and say the results.

f3d09597bbd4220e033b5ecc26e166b8

Also seen in the image that "play" request exists, but protocol cannot recognize it...

TrueTrAxeX commented 10 years ago

I check it in ubuntu 64x. Bug still exists. Can you give your client version? (my client downloaded from this repository)

TrueTrAxeX commented 10 years ago

6c0c4b86b6e0441f8f65e1b4da32aea2

In debugger this param also exists

TrueTrAxeX commented 10 years ago

I think this bug is related to encoding package into AMF

dominick commented 10 years ago

Based on the screenshot of your eclipse debug session, it looks like the entire string is being encoded which leaves me confused as to why wireshark isn't seeing the data properly. What version of wireshark are you using? Do you think it could be a problem with wireshark? Does the stream get loaded up on the Red5 side?

TrueTrAxeX commented 10 years ago

Stream not loading on red5 server... And also your red5 server not support aggregate messages!

TrueTrAxeX commented 10 years ago

Now i use librtmp C++ library for me tasks and all it's ok... Red5 is still very raw ...

mondain commented 10 years ago

Just to let you all know Red5 does support Aggregate messages.