-
Notifications
You must be signed in to change notification settings - Fork 234
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
RtmpClient.read returns -1 rather than throwing when no connection #69
Comments
Hi, |
Yes, if |
Hi @mekya - could you please provide me an email adress to send the stream url I'm using. |
Hi @mekya - Will the proposed solution will be implemented in the upcoming releases? |
vavadhani
referenced
this issue
in ittiam-systems/LibRtmp-Client-for-Android
Jun 21, 2019
Propagate the error codes from RTMP read to nativeRead function and return errors as expected. This commit also corrects exception throws from nativeRead.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Reproduction steps:
RtmpClient.read
, as normal. @jinfanzon can provide you with such a stream via email.RtmpClient.read
call ends up returning -1, which is documented to mean that the stream is complete.It looks to me like returning -1 in this case is incorrect. In ExoPlayer, it causes us to assume the stream has ended and transition the player to its ended state. The
RtmpClient.read
call should either throw an exception (in which case the player will retry) or read some valid data (if possible, depending on when the method returns relative to when airplane mode is toggled off again). Please could you take a look?The text was updated successfully, but these errors were encountered: