Merge remote-tracking branch 'origin/master' into AutoDetectLongPolli…

…ngEnabledByDefault
firebase · dconeybe · May 8, 2023 · Apr 20, 2023 · Apr 20, 2023 · Apr 20, 2023
commit e58f70be9f2a9d67f1564ff96ec5ef597d2933c4
diff --git a/packages/firestore/src/lite-api/settings.ts b/packages/firestore/src/lite-api/settings.ts
@@ -34,6 +34,18 @@ import { validateIsNotUsedTogether } from '../util/input_validation';
 export const DEFAULT_HOST = 'firestore.googleapis.com';
 export const DEFAULT_SSL = true;
 
+// The minimum long-polling timeout is hardcoded on the server. The value here
+// should be kept in sync with the value used by the server, as the server will
+// silently ignore a value below the minimum and fall back to the default.
+// Googlers see b/266868871 for relevant discussion.
+const MIN_LONG_POLLING_TIMEOUT_SECONDS = 5;
+
+// No maximum long-polling timeout is configured in the server, and defaults to
+// 30 seconds, which is what Watch appears to use.
+// Googlers see b/266868871 for relevant discussion.
+const MAX_LONG_POLLING_TIMEOUT_SECONDS = 30;
+
+// Whether long-polling auto-detected is enabled by default.
 const DEFAULT_AUTO_DETECT_LONG_POLLING = true;
 
 /**