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

Camera Streamer UDP Ghost Stream #135

Open
Xaver106 opened this issue Jun 29, 2023 · 4 comments
Open

Camera Streamer UDP Ghost Stream #135

Xaver106 opened this issue Jun 29, 2023 · 4 comments
Labels
bug Something isn't working

Comments

@Xaver106
Copy link

What happened

Camera Streamer currently has an isse, where a stream keeps running after it has been closed. This can overwhelm older Hardware quite quickly. In my case even causing me to no longer be able to reach my mainsail instance.
ayufan/camera-streamer#55

What did you expect to happen

A mention about this on the Documentation.

How to reproduce

Explainewd in the Linked Bug Report: ayufan/camera-streamer#55

Additional information

It is probably best to either not reccomend the camera Streamer as default or add a warning.

@Xaver106 Xaver106 added the bug Something isn't working label Jun 29, 2023
@Xaver106
Copy link
Author

Ustreamer seems to still depend on the legacy camera support. Not using the WebRTC Stream option and sticking to the MJPEG-streamer seems to not trigger the Ghost Stream

@jpiccari
Copy link

Ghost streams are caused by lack of proper shutdown of the stream by the clients. Chrome, Edge, Safari, etc do not properly signal closing the peer connection and instead just stop receiving communication. For mobile devices this happens quite frequently due to locking/unlocking a phone. I have a fix that adds a keep-alive type mechanism to camera-streamer but it will also require updates to Mainsail as it has its own front-end code displaying the stream.

I haven't looking into the rtsp implementation used within Mainsail/camera-streamer but that might be another viable workaround.

@meteyou
Copy link
Member

meteyou commented Jul 29, 2023

@jpiccari Mainsail doesn't have a rtsp stream client right now, but feel free to contact me, if you need help to implement the keep-alive mechanism in the webrtc client in Mainsail.

@jpiccari
Copy link

jpiccari commented Aug 5, 2023

I can submit a PR with the changes I have in my local mainsail. It will be dependent on camera-streamer being updated (also need to submit a PR over there) and frankly, camera-streamer is just not very reliable (webrtc or otherwise) in my experience running on a Raspberry Pi 3B+ so even with the fix I think it is probably better for users to look to ustreamer + janus-gateway for webrtc streaming (although I haven't been able to set that up yet, anyone point me to some docs?).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants