enforce max size, not just default max size

getsentry · Apr 18, 2024 · 77c675d · 77c675d
1 parent d439404
commit 77c675d
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/packages/replay-canvas/src/canvas.ts b/packages/replay-canvas/src/canvas.ts
@@ -55,13 +55,22 @@ const CANVAS_QUALITY = {
 };
 
 const INTEGRATION_NAME = 'ReplayCanvas';
+const DEFAULT_MAX_CANVAS_SIZE = 1280;
 
 /** Exported only for type safe tests. */
 export const _replayCanvasIntegration = ((options: Partial<ReplayCanvasOptions> = {}) => {
+  const [maxCanvasWidth, maxCanvasHeight] = options.maxCanvasSize || [];
   const _canvasOptions = {
     quality: options.quality || 'medium',
     enableManualSnapshot: options.enableManualSnapshot,
-    maxCanvasSize: options.maxCanvasSize || [1280, 1280],
+    maxCanvasSize: [
+      maxCanvasWidth ?
+        Math.max(maxCanvasWidth, DEFAULT_MAX_CANVAS_SIZE)
+      : DEFAULT_MAX_CANVAS_SIZE,
+      maxCanvasHeight ?
+        Math.max(maxCanvasHeight, DEFAULT_MAX_CANVAS_SIZE)
+        : DEFAULT_MAX_CANVAS_SIZE
+    ],
   };
 
   let canvasManagerResolve: (value: CanvasManager) => void;