Skip to content

Add nonce to scripts if configured#1129

Merged
jwag956 merged 2 commits into
pallets-eco:mainfrom
ahanak:script-nonce
Oct 11, 2025
Merged

Add nonce to scripts if configured#1129
jwag956 merged 2 commits into
pallets-eco:mainfrom
ahanak:script-nonce

Conversation

@ahanak

@ahanak ahanak commented Oct 10, 2025

Copy link
Copy Markdown
Contributor

This adds the SECURITY_SCRIPT_NONCE_KEY config option.

A usage example can found under tests/view_scaffold.py

Fixes #1127

Fixing:
$ sphinx-build docs docs/_build/html
'Flask-Security' must be installed to build the documentation.
This adds the SECURITY_SCRIPT_NONCE_KEY config option.

A usage example can found under tests/view_scaffold.py
@ahanak ahanak changed the title Script nonce Add nonce to scripts if configured Oct 10, 2025
@codecov

codecov Bot commented Oct 10, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.52%. Comparing base (bf353fd) to head (dccf6c2).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1129   +/-   ##
=======================================
  Coverage   98.52%   98.52%           
=======================================
  Files          40       40           
  Lines        5015     5015           
=======================================
  Hits         4941     4941           
  Misses         74       74           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jwag956 jwag956 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Absolutely perfect - thanks!

@jwag956
jwag956 merged commit 7438886 into pallets-eco:main Oct 11, 2025
16 checks passed
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Oct 28, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Support for Content-Security-Policy CSP

3 participants