Skip to content

Fix style csp#17

Merged
pedrordgs merged 1 commit intomasterfrom
fix/style-csp
Jan 12, 2026
Merged

Fix style csp#17
pedrordgs merged 1 commit intomasterfrom
fix/style-csp

Conversation

@pedrordgs
Copy link
Owner

@pedrordgs pedrordgs commented Jan 12, 2026

Description

Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📝 Documentation update
  • 🎨 Style/UI update (no functional changes)
  • ♻️ Code refactoring (no functional changes)
  • ⚡ Performance improvement
  • ✅ Test update

Related Issues

Fixes #
Related to #

Screenshots

Before

After

Additional Context

Deployment Notes


Summary by CodeRabbit

  • Chores

    • Version updated to 1.1.1
  • Configuration Updates

    • Content-Security-Policy for inline styles has been modified

✏️ Tip: You can customize this high-level summary in your review settings.

@pedrordgs pedrordgs self-assigned this Jan 12, 2026
@coderabbitai
Copy link

coderabbitai bot commented Jan 12, 2026

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

Version bumped to 1.1.1. Content-Security-Policy in proxy.ts modified to unconditionally include 'unsafe-inline' in style-src instead of using conditional nonce-based logic. All other CSP directives remain unchanged.

Changes

Cohort / File(s) Summary
Version Update
package.json
Version bumped from 1.0.1 to 1.1.1
Security Configuration
proxy.ts
Modified style-src CSP directive to always include 'unsafe-inline' instead of conditional nonce-based source handling

Possibly related PRs

  • Improve security headers #16: Added nonce-based CSP generation logic that is now being replaced by this PR's unconditional 'unsafe-inline' approach in style-src.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A version hops forward, now 1.1.1!
Styles flow free without nonce to fret,
Inline at last—the constraint is undone,
Simple and secure, our CSP is set!

✨ Finishing touches
  • 📝 Generate docstrings

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between cda16c0 and 6e7fa46.

📒 Files selected for processing (2)
  • package.json
  • proxy.ts

Comment @coderabbitai help to get the list of available commands and usage tips.

@pedrordgs pedrordgs merged commit 42eb08a into master Jan 12, 2026
3 of 4 checks passed
@pedrordgs pedrordgs deleted the fix/style-csp branch January 12, 2026 23:17
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