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

Support RTSP stream #11

Open
arsaboo opened this issue Mar 20, 2019 · 4 comments
Open

Support RTSP stream #11

arsaboo opened this issue Mar 20, 2019 · 4 comments

Comments

@arsaboo
Copy link

arsaboo commented Mar 20, 2019

With the upcoming stream component (in 0.90) in Home Assistant that only supports RTSP streams without transcoding, I was wondering if we should consider providing an RTSP stream.

Surveillance Station supports RTSP stream and we should consider implementing that.

@pergolafabio
Copy link

But, why not make a camera then in HA with the rtsp path? Then you have it already

@arsaboo
Copy link
Author

arsaboo commented Mar 20, 2019

Sure, I can manually add individual cameras and maybe that is the way to go. I was thinking that the cameras that are added with the component could use RTSP.

@arsaboo arsaboo closed this as completed Mar 20, 2019
@arsaboo arsaboo reopened this Mar 20, 2019
@hunterjm
Copy link

I think this is a valid issue. Currently, _video_stream_url is hardcoded to the MJPEG url. The documentation specifies a GetLiveViewPath method in section 2.3.4.18 which this method could utilize. That would give us the ability to pass in the already defined video_format and request the stream path we want from Synology.

The advantage would be to continue to use the Synology component that adds all of the cameras instead of defining each as a "Generic IP Camera" in the configuration. If this library is updated, I can easily add support in Home Assistant.

@arsaboo
Copy link
Author

arsaboo commented Mar 21, 2019

Looks like @alekslyse did that in his (now closed) PR. Wondering if we should re-open that PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants