Skip to content

Conversation

@JeffScattini-amplitude
Copy link
Collaborator

@JeffScattini-amplitude JeffScattini-amplitude commented Dec 9, 2025

Add comprehensive CSP sections to both session-replay-standalone-sdk.md and session-replay-plugin.md, including:

  • Required CSP directives (script-src, connect-src, worker-src)
  • Directive reference table with domains and descriptions
  • API endpoints table for US and EU data centers
  • Example CSP headers for both regions

Resolves DOC-931


Note

Add CSP guidance (directives, endpoints, example headers) to Session Replay Plugin and Standalone SDK docs.

  • Docs (Session Replay):
    • Plugin (session-replay-plugin.md):
      • Add CSP section with required directives (script-src, connect-src, worker-src).
      • Include directive reference table, API endpoints (US/EU), and example CSP headers.
      • Add tip to include custom configServerUrl/trackServerUrl domains in connect-src.
    • Standalone SDK (session-replay-standalone-sdk.md):
      • Mirror CSP guidance: required directives, reference table, API endpoints (US/EU), example headers, and custom endpoint tip.

Written by Cursor Bugbot for commit 26b2775. This will update automatically on new commits. Configure here.

Add comprehensive CSP sections to both session-replay-standalone-sdk.md and
session-replay-plugin.md, including:
- Required CSP directives (script-src, connect-src, worker-src)
- Directive reference table with domains and descriptions
- API endpoints table for US and EU data centers
- Example CSP headers for both regions

Resolves DOC-931
@vercel
Copy link

vercel bot commented Dec 9, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
amplitude-docs Canceled Canceled Dec 17, 2025 6:01pm

Copy link
Member

@markzegarelli markzegarelli left a comment

Choose a reason for hiding this comment

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

few suggestions.

Also, if this content is identical, we should make it a partial so we can single source.


#### Example CSP header

For US data center:
Copy link
Member

Choose a reason for hiding this comment

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

including data center might be confusing here.

Suggested change
For US data center:
For the US region:

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants