Skip to content

chore(security): harden Airtable contact embed on /contact#195

Merged
dandersonsw merged 3 commits into
mainfrom
2025-08-29_fix_contact_sandbox
Aug 29, 2025
Merged

chore(security): harden Airtable contact embed on /contact#195
dandersonsw merged 3 commits into
mainfrom
2025-08-29_fix_contact_sandbox

Conversation

@wz-gsa

@wz-gsa wz-gsa commented Aug 29, 2025

Copy link
Copy Markdown
Contributor

Changes proposed in this pull request:

  • Replace raw Airtable iframe with a secured embed:
    • sandbox="allow-forms allow-scripts" (isolate third-party content)
    • referrerpolicy="origin" (Airtable sees only https://cloud.gov as referrer)
    • Deny powerful features via iframe allow (permissions policy)
    • Preserve loading.gif UX (cleared on onload)

security considerations

  • sandbox isolates the embed and enables only what a form needs (forms + scripts).
  • The iframe allow attribute further restricts powerful features (Permissions Policy) on top of any page-level policy.
  • referrerpolicy="origin" lets Airtable confirm traffic origin (e.g., https://cloud.gov) without leaking paths/params

@wz-gsa wz-gsa self-assigned this Aug 29, 2025

@dandersonsw dandersonsw left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@dandersonsw
dandersonsw merged commit d5c7ca7 into main Aug 29, 2025
3 checks passed
@dandersonsw
dandersonsw deleted the 2025-08-29_fix_contact_sandbox branch August 29, 2025 18:49
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.

2 participants