fix: can not play video in iframe

[hyoban]

Description

PR Type

• Feature
• Bugfix
• Hotfix
• Other (please describe):

Screenshots (if UI change)

Demo Video (if new feature)

Linked Issues

close #2996

Additional context

Changelog

• I have updated the changelog/next.md with my changes.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
follow	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 11, 2025 3:35am
follow-external-ssr	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 11, 2025 3:35am

[follow-reviewer-bot]

Suggested PR Title:

refactor(headers): improve createBuildSafeHeaders structure

Change Summary:
Refactor createBuildSafeHeaders for improved handling of headers, allowing for no fallback option and enhanced structure in various components. Updated multiple files to align with this new structure, ensuring consistency across the application.

Code Review:

Code Review

The following are issues that require a change request:

---

File: packages/utils/src/headers.ts

Lines: 7-15

• The parameter noFallbak is a typo and should likely be corrected to noFallback for consistency and clarity. Using clear, correctly spelled parameter names is critical for code readability and understanding by future maintainers.

Recommended Change:

({
    webUrl,
    selfRefererMatches,
    noFallback,  // corrected typo
}: {
    webUrl: string
    selfRefererMatches: string[]
    noFallback?: boolean  // corrected typo
}) =>

Update all occurrences of the misspelled parameter (noFallbak) throughout the file to reflect the corrected spelling.

---

File: apps/desktop/src/main/src/index.ts

Line: 38

• The new object key noFallbak passed to createBuildSafeHeaders also contains the same typo as noted in headers.ts. This will cause a bug, as createBuildSafeHeaders will not recognize noFallbak due to the incorrect spelling.

Recommended Change:

noFallback: true,  // corrected typo

---

Files: All other files (Image.tsx, ProxiedImage.tsx, SearchFeed.tsx)

• These files use createBuildSafeHeaders but do not pass the new noFallbak parameter. If the absence of this optional parameter was intentional, no change is needed. Otherwise, clarify or add noFallback explicitly to ensure consistent configurations with other usages.

---

Summary of Required Changes

1. Correct the noFallbak typo to noFallback in the following places:

   • packages/utils/src/headers.ts (function parameter and internal handling).
   • apps/desktop/src/main/src/index.ts (parameter passed to the function).

2. Ensure consistent handling of the noFallback parameter across all createBuildSafeHeaders calls. Either explicitly add it where missing or confirm intentional exclusion.

---

Let me know if you need further clarification!