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

Clean HASS Urls that come from UniFi Protect #867

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ViViDboarder
Copy link

This automatically applies the recommended configuration for UniFi cameras. This could possibly be moved to the Home Assistant component when passing the URL, but I'm starting here to get your thoughts.

Closes #81

This automatically applies the recommended configuration for UniFi cameras.

Closes AlexxIT#81
@AlexxIT
Copy link
Owner

AlexxIT commented Jan 10, 2024

I don't think this is good solution.

  1. RTSPX should't be necessary, because latest go2rtc won't check TLS for IP-address as host.
  2. Removing enableSrtp necessary. But I don't think this should be automatically. It's better to add SRTP support, but I don't have access to any UniFi camera

@ViViDboarder
Copy link
Author

It didn’t seem that you were interested in adding SRTP support because it was only useful for UDP and go2rtc only used TCP. Is the scenario changed? I am not very familiar with the protocol, but I can poke around at it.

@AlexxIT
Copy link
Owner

AlexxIT commented Jan 10, 2024

You right. I think it's unnecessary to use SRTP over HTTPS. But looks like this is default option for UniFi.

@AlexxIT AlexxIT added the doubt label Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support SRTP for UniFi RTSPS cameras
2 participants