From 6ee88f8a7d55f4095d9664c5ef7323efb986a72a Mon Sep 17 00:00:00 2001
From: Gabe Kangas <gabek@real-ity.com>
Date: Mon, 25 Apr 2022 14:52:33 -0700
Subject: [PATCH] Throw away latency values that seem invalid

---
 webroot/js/metrics/playback.js | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/webroot/js/metrics/playback.js b/webroot/js/metrics/playback.js
index 99de95b77..82103509c 100644
--- a/webroot/js/metrics/playback.js
+++ b/webroot/js/metrics/playback.js
@@ -1,5 +1,6 @@
 import { URL_PLAYBACK_METRICS } from '../utils/constants.js';
 const METRICS_SEND_INTERVAL = 10000;
+const MAX_VALID_LATENCY_SECONDS = 40; // Anything > this gets thrown out.
 
 class PlaybackMetrics {
   constructor(player, videojs) {
@@ -174,6 +175,12 @@ class PlaybackMetrics {
       const segmentTime = segment.dateTimeObject.getTime();
       const now = new Date().getTime();
       const latency = now - segmentTime;
+
+      // Throw away values that seem invalid.
+      if (latency < 0 || latency / 1000 >= MAX_VALID_LATENCY_SECONDS) {
+        return;
+      }
+
       this.trackLatency(latency);
     } catch (err) {
       console.warn(err);