Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
❗ Important Announcement
Click here for more details:
🚧 Temporary Delay in Feature Requests and Pull Request Reviews
At this time, we may be slower to respond to new feature requests and review pull requests. Existing requests and PRs will remain in the backlog but may not be prioritized immediately.
We appreciate your patience and understanding as we continue to improve Uptime Kuma.
🚫 Please Avoid Unnecessary Pinging of Maintainers
We kindly ask you to refrain from pinging maintainers unless absolutely necessary. Pings are reserved for critical/urgent pull requests that require immediate attention.
Why: Reserving pings for urgent matters ensures maintainers can prioritize critical tasks effectively.
📋 Overview
Provide a clear summary of the purpose and scope of this pull request:
This PR (once complete) addresses RTSP (Real Time Streaming Protocol)) support #2851 by implementing a new monitoring type: RTSP. The goal is to give folks the ability to monitor their RTSP cameras and alert them when they go down.
The exact implementation seems fairly straight-forwarded (see the latest comment in the linked issue) but I wanted to run an idea past y'all before I implemented this: what about two "levels" of checking: simple (look for the
RTSP/1.0 200 OK
text) and "deep" (use ffprobe to look for actual video data). This would be implemented as a single RTSP monitoring type with a "simple | deep" toggle or "deep" checkbox (whichever is preferred). If we went with implementing the "deep" option we would need to add ffprobe to our list of deps I think (for the docker container). I'm unsure if we could install ffprobe just-in-time. Let me know what y'all think. Looking forward to helping out here.What features or functionality does this pull request introduce or enhance?
A new RTSP monitoring type will be added
🔄 Changes
🛠️ Type of change
🔗 Related Issues
📄 Checklist *
📷 Screenshots or Visual Changes
UP
DOWN
ℹ️ Additional Context
Provide any relevant details to assist reviewers in understanding the changes.
Click here for more details:
Key Considerations:
Provide details here
💬 Requested Feedback
Mention documents needing feedback here