Skip to content

fix(apps): make external-link sandbox escape cache-safe#102

Closed
hamzamerzic wants to merge 1 commit into
mobius-os:mainfrom
hamzamerzic:fix/app-link-sandbox-escape
Closed

fix(apps): make external-link sandbox escape cache-safe#102
hamzamerzic wants to merge 1 commit into
mobius-os:mainfrom
hamzamerzic:fix/app-link-sandbox-escape

Conversation

@hamzamerzic

@hamzamerzic hamzamerzic commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • allow app-opened external links to escape the inherited iframe sandbox
  • bump the offline app-shell cache version so existing installations receive the policy change
  • preserve the current CSP and Cache-Control headers on conditional 304 responses
  • add frontend and backend regressions for the complete upgrade path

Review note

The original sandbox flag was directionally correct but incomplete because cached shells could retain the old policy. This revision closes that deployment boundary.

Validation

  • 36 focused frontend tests passed
  • 24 focused backend tests passed
  • production build passed
  • full frontend and backend suites passed through the privacy pre-push gate

Co-authored-by: Möbius Agent <mobius-agent@users.noreply.github.com>
@hamzamerzic
hamzamerzic force-pushed the fix/app-link-sandbox-escape branch from be5b219 to eba0699 Compare July 22, 2026 02:49
@hamzamerzic hamzamerzic changed the title Let external app links escape inherited sandbox restrictions fix(apps): make external-link sandbox escape cache-safe Jul 22, 2026
@hamzamerzic

Copy link
Copy Markdown
Collaborator Author

Superseded by #108, which includes this reviewed sandbox/cache/304 work plus the hosted-CI correction to the offline cache revision test. #108 passed the full fresh suite and merged as 5baaf65.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant