refactor external events handler

[skrashevich]

separate logic and API handlers. It's now possible to create "external" events without requesting the http server.

For example, possible changes on audio-events:

---
 frigate/events/audio.py | 16 +++++++---------
 1 file changed, 7 insertions(+), 9 deletions(-)

diff --git a/frigate/events/audio.py b/frigate/events/audio.py
index 0decb001b..f1354e633 100644
--- a/frigate/events/audio.py
+++ b/frigate/events/audio.py
@@ -21,6 +21,7 @@
     AUDIO_SAMPLE_RATE,
     CACHE_DIR,
 )
+from frigate.events.external import create_event, end_event
 from frigate.ffmpeg_presets import parse_preset_input
 from frigate.log import LogPipe
 from frigate.object_detection import load_labels
@@ -184,10 +185,7 @@ def handle_detection(self, label: str, score: float) -> None:
                 "last_detection"
             ] = datetime.datetime.now().timestamp()
         else:
-            resp = requests.post(
-                f"http:https://127.0.0.1:5000/api/events/{self.config.name}/{label}/create",
-                json={"duration": None},
-            )
+            resp = create_event(self.config.name, label, {"duration": None})
 
             if resp.status_code == 200:
                 event_id = resp.json()[0]["event_id"]
@@ -205,11 +203,11 @@ def expire_detections(self) -> None:
                 now - detection.get("last_detection", now)
                 > self.config.audio.max_not_heard
             ):
-                resp = requests.put(
-                    f"http:https://127.0.0.1/api/events/{detection['id']}/end",
-                    json={
-                        "end_time": detection["last_detection"]
-                        + self.config.record.events.post_capture
+                resp = end_event(
+                    detection["id"],
+                    {
+                        "end_time": detection["last_detection"] * 1
+                        + self.config.record.events.post_capture * 1
                     },
                 )
                 if resp.status_code == 200:

[netlify]

✅ Deploy Preview for frigate-docs canceled.

Name	Link
🔨 Latest commit	99f23c4
🔍 Latest deploy log	https://app.netlify.com/sites/frigate-docs/deploys/64999565ed9afb00087bc051

[NickM-27]

I don't think we want Flask being used outside of the http module, this breaks separation of concerns

[NickM-27]

This also wouldn't work for the audio events feature because audio events are handled in a separate process so current_app wouldn't be set.

I don't see any problem with the way things are currently done.