Skip to content

Optimize ring #1745

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

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Optimize ring #1745

wants to merge 10 commits into from

Conversation

seydx
Copy link
Contributor

@seydx seydx commented May 21, 2025

Ring Client Improvements for go2rtc

I've implemented several significant improvements to the Ring client integration in go2rtc.

Improvements

  • Significantly faster stream loading: I've optimized the streaming process by eliminating unnecessary API fetching prior to stream initialization, resulting in noticeably quicker stream opening times.

  • Enhanced code readability: I've thoroughly refactored the Ring client code to improve maintainability and facilitate easier future development.

  • Performance optimizations: I've implemented various improvements throughout the client to enhance overall performance and responsiveness.

Breaking Change Notice

Important: This update includes a breaking change for users with existing Ring camera configurations. To ensure proper functionality after updating:

  1. Remove your current Ring cameras from go2rtc
  2. Re-add them using the Add function

This one-time reconfiguration is necessary to implement the improved connection method and ensure you benefit from the faster streaming capabilities.

@AlexxIT AlexxIT self-assigned this May 21, 2025
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

Successfully merging this pull request may close these issues.

2 participants