Skip to content

fix: log variable substitution, CSRF exclusions, and README overhaul#9

Merged
audunru merged 1 commit intomainfrom
fix/log-variable-substitution
May 3, 2026
Merged

fix: log variable substitution, CSRF exclusions, and README overhaul#9
audunru merged 1 commit intomainfrom
fix/log-variable-substitution

Conversation

@audunru
Copy link
Copy Markdown
Owner

@audunru audunru commented May 3, 2026

Summary

  • Use PHP string interpolation in LogCspViolation and LogReport log messages (fixes variable not being substituted in log output)
  • Exclude all CSRF middleware variants from the reports route (not just the default one)
  • Restructure README: add GitHub Actions + Coveralls badges, replace separate "Listening for events" / "Built-in listeners" sections with a single "Getting started" section using Event::listen() in AppServiceProvider::boot(), move DTO/event reference tables to a new "Reference" section

Closes #4

Test plan

  • Existing test suite passes (composer test)
  • Log output for CSP violations and generic reports contains the expected interpolated values
  • README renders correctly on GitHub

🤖 Generated with Claude Code

…ference section

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@audunru audunru force-pushed the fix/log-variable-substitution branch from 752a213 to d7d734a Compare May 3, 2026 08:21
@audunru audunru merged commit 34426aa into main May 3, 2026
2 checks passed
@audunru audunru deleted the fix/log-variable-substitution branch May 3, 2026 08:21
@audunru
Copy link
Copy Markdown
Owner Author

audunru commented May 3, 2026

🎉 This PR is included in version 2.0.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update README.md

1 participant