Skip to content

fix: Cal Video fails to load in Firefox with Enhanced Tracking Protection#28714

Open
sudoKrishna wants to merge 7 commits into
calcom:mainfrom
sudoKrishna:fix/firefox-video-loading
Open

fix: Cal Video fails to load in Firefox with Enhanced Tracking Protection#28714
sudoKrishna wants to merge 7 commits into
calcom:mainfrom
sudoKrishna:fix/firefox-video-loading

Conversation

@sudoKrishna
Copy link
Copy Markdown

Fixes #28473

Issue

Cal Video fails to load in Firefox when Enhanced Tracking Protection is enabled.
Users see an infinite loading screen.

Fix

  • Added fallback UI when Daily iframe fails to load
  • Added timeout detection
  • Prevent infinite loading state

Testing

  • Tested locally
  • Verified fallback UI appears when video fails
  • Verified video loads normally in supported browsers

@sudoKrishna sudoKrishna requested review from a team as code owners April 2, 2026 19:23
@github-actions github-actions Bot added the 🐛 bug Something isn't working label Apr 2, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 2, 2026

Hey there and thank you for opening this pull request! 👋🏼

We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted.

Details:

Unknown release type "Fix" found in pull request title "Fix: Cal Video fails to load in Firefox with Enhanced Tracking Protection". 

Available types:
 - feat: A new feature
 - fix: A bug fix
 - docs: Documentation only changes
 - style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
 - refactor: A code change that neither fixes a bug nor adds a feature
 - perf: A code change that improves performance
 - test: Adding missing tests or correcting existing tests
 - build: Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)
 - ci: Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
 - chore: Other changes that don't modify src or test files
 - revert: Reverts a previous commit

@github-actions github-actions Bot added the ❗️ .env changes contains changes to env variables label Apr 2, 2026
Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

3 issues found across 4 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name=".env.example">

<violation number="1" location=".env.example:74">
P1: Hardcoded default secrets in .env.example create shared credentials for deployments that copy the defaults, undermining auth/session signing and encryption.</violation>
</file>

<file name="apps/web/modules/videos/views/videos-single-view.tsx">

<violation number="1" location="apps/web/modules/videos/views/videos-single-view.tsx:156">
P1: Failure timeout is unconditional and not cancelled on successful call join, so it can incorrectly switch UI to `videoFailed` after 8s.</violation>

<violation number="2" location="apps/web/modules/videos/views/videos-single-view.tsx:189">
P2: Failure fallback UI can render without destroying the active Daily call, potentially leaking background media/session resources.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review, or fix all with cubic.

Comment thread .env.example Outdated
Comment thread apps/web/modules/videos/views/videos-single-view.tsx
Comment thread apps/web/modules/videos/views/videos-single-view.tsx
@sudoKrishna sudoKrishna changed the title Fix: Cal Video fails to load in Firefox with Enhanced Tracking Protection fix: Cal Video fails to load in Firefox with Enhanced Tracking Protection Apr 3, 2026
@sudoKrishna
Copy link
Copy Markdown
Author

Hi @calcom/foundation Could a maintainer please approve the workflows to run? This PR fixes #28473 where Firefox users with Enhanced Tracking Protection see an infinite loading screen instead of joining the video call. All review issues
( timeout bug, call cleanup) have been addressed. Thanks!

@github-actions
Copy link
Copy Markdown
Contributor

This PR has been marked as stale due to inactivity. If you're still working on it or need any help, please let us know or update the PR to keep it active.

@github-actions github-actions Bot added the Stale label Apr 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 bug Something isn't working ❗️ .env changes contains changes to env variables size/XL Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cal Video fails to load in Firefox 148 with Enhanced Tracking Protection enabled

1 participant