Skip to content
This repository has been archived by the owner on Sep 30, 2021. It is now read-only.

rtmp: recovery from bad url does not work #589

Closed
phloxic opened this issue Dec 1, 2013 · 7 comments
Closed

rtmp: recovery from bad url does not work #589

phloxic opened this issue Dec 1, 2013 · 7 comments
Assignees
@anssip
Milestone
6.0 - next major ...

Comments

@phloxic
Copy link
Contributor

phloxic commented Dec 1, 2013

http://demos.flowplayer.org/scripting/recover.html?engine=flash does not work if the flash sources would be delivered via RTMP.
Demo to follow.
@ghost ghost assigned anssip Dec 1, 2013
@phloxic
Copy link
Contributor Author

phloxic commented Dec 1, 2013

Demo at http://demos.flowplayer.org/bug/589.html

@nnarhinen nnarhinen added this to the 5.4.7 milestone Apr 14, 2014
@nnarhinen nnarhinen modified the milestones: 5.6.0, 6.0 - next major release Sep 16, 2014
@anssip
Copy link
Contributor

anssip commented Nov 13, 2014

It does not seem to detect in Flash if the stream is invalid and cannot be played. Checking again maybe later if there's some sane way to implement this.

@nnarhinen
Copy link
Member

@blacktrash does FP Flash detect this correctly?

@phloxic
Copy link
Contributor Author

phloxic commented Feb 23, 2015

Yes: http://flowplayer.blacktrash.org/test/rtmp-recover.html correctly reports error 200 "stream not found".
Flowplayer HTML5 simply hangs, I also tried with full rtmp url.

anssip added a commit that referenced this issue Apr 27, 2015
@anssip
Make it fire error code 4 if the stream is not found with RTMP. fixes #…
18dfb4f 
…589
@anssip
Copy link
Contributor

anssip commented Apr 27, 2015

fixed. Now fires an error with code 4 when the stream is not found.

@anssip anssip closed this as completed Apr 27, 2015
@phloxic
Copy link
Contributor Author

phloxic commented Apr 27, 2015

There is an error, but the API only receives undefined as error, not an error object.

@phloxic
Copy link
Contributor Author

phloxic commented Apr 28, 2015

@anssip - sorry for the noise, I probably tested while the change was not yet propagated. All is fine.

anssip added a commit that referenced this issue May 21, 2015
@anssip @nnarhinen
Make it fire error code 4 if the stream is not found with RTMP. fixes #…
6878eb4 
…589
@foliovision foliovision mentioned this issue Jun 27, 2022
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@anssip anssip

Labels
None yet
Projects
None yet
Milestone
6.0 - next major release
Development

No branches or pull requests
3 participants
@phloxic @nnarhinen @anssip