Skip to content

Conversation

@StanBarrows
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings July 30, 2025 00:13
@StanBarrows StanBarrows merged commit 13b96a1 into production Jul 30, 2025
5 checks passed
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes the @cspNonce directive from the Fathom analytics script tag in the Blade template.

  • Removes CSP (Content Security Policy) nonce attribute from the Fathom analytics script

@if (in_array(app()->environment(), $environments) && $siteId)
<script @cspNonce src="{{ $url }}" data-site="{{ $siteId }}" type="text/javascript" defer></script>
@endif
<script src="{{ $url }}" data-site="{{ $siteId }}" type="text/javascript" defer></script>
Copy link

Copilot AI Jul 30, 2025

Choose a reason for hiding this comment

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

Removing the @cspNonce directive weakens Content Security Policy protection. Without the nonce, this script may be blocked by CSP headers or could potentially allow unauthorized script execution. Consider keeping the nonce for security compliance.

Suggested change
<script src="{{ $url }}" data-site="{{ $siteId }}" type="text/javascript" defer></script>
<script src="{{ $url }}" data-site="{{ $siteId }}" type="text/javascript" nonce="{{ cspNonce() }}" defer></script>

Copilot uses AI. Check for mistakes.
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