Skip to content

Security improvements: flexible verification modes and CSRF meta tag support#4

Merged
xentixar merged 3 commits into
diffyne:mainfrom
xentixar:security-improvements
Dec 8, 2025
Merged

Security improvements: flexible verification modes and CSRF meta tag support#4
xentixar merged 3 commits into
diffyne:mainfrom
xentixar:security-improvements

Conversation

@xentixar
Copy link
Copy Markdown
Contributor

@xentixar xentixar commented Dec 8, 2025

  • Added flexible security modes: property-updates (default), strict, and none
  • Updated both HTTP and WebSocket controllers with consistent verification logic
  • Added CSRF meta tag support via @diffyneStyles directive
  • Fixed state tampering issues with diff:model
  • Improved form submission handling while maintaining security
  • Updated documentation with new security configuration
  • Cleaned up verbose comments and unused code

…support

- Added flexible security modes: property-updates (default), strict, and none
- Updated both HTTP and WebSocket controllers with consistent verification logic
- Added CSRF meta tag support via @diffyneStyles directive
- Fixed state tampering issues with diff:model
- Improved form submission handling while maintaining security
- Updated documentation with new security configuration
- Cleaned up verbose comments and unused code
- Removed unnecessary whitespace in the DiffyneController and WebSocket DiffyneController for improved code readability.
- Added PHPDoc comments to the normalizeStateForVerification method to enhance code documentation and clarity on parameter types and return values.
- Minor whitespace adjustments in the DiffyneServiceProvider for consistency.
@xentixar xentixar added security Security vulnerability or security-related issue and removed bug Something isn't working labels Dec 8, 2025
- Introduced a CI approval job to ensure that checks only proceed after explicit approval.
- Updated static analysis, format check, and test jobs to depend on the approval step, improving control over the CI process.
@xentixar xentixar merged commit bccac9c into diffyne:main Dec 8, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

security Security vulnerability or security-related issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant