Skip to content

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

Open
sudoKrishna wants to merge 7 commits intocalcom: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 intocalcom: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.

@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!

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

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