Skip to content

Conversation

@hyperpolymath
Copy link
Owner

This commit enhances the plugin's security posture by integrating patterns from the php-aegis and sanctify-php security libraries.

Changes:

  • Add WPCM_Security class (PHP 7.4+ compatible security layer)
  • Add declare(strict_types=1) to all PHP files
  • Add SPDX license headers to all PHP files
  • Fix REST API limit parameter sanitization
  • Add security-analysis.yml GitHub workflow
  • Add docs/SECURITY-INTEGRATION.md with integration report
  • Update composer.json with security scripts and suggestions

The integration report documents:

  • PHP version incompatibility (php-aegis requires 8.1+, plugin needs 7.4+)
  • sanctify-php is a Haskell build tool, not a runtime PHP library
  • Recommendations for upstream improvements to both libraries

Version bumped to 1.1.0 to reflect security enhancements.

This commit enhances the plugin's security posture by integrating patterns
from the php-aegis and sanctify-php security libraries.

Changes:
- Add WPCM_Security class (PHP 7.4+ compatible security layer)
- Add declare(strict_types=1) to all PHP files
- Add SPDX license headers to all PHP files
- Fix REST API limit parameter sanitization
- Add security-analysis.yml GitHub workflow
- Add docs/SECURITY-INTEGRATION.md with integration report
- Update composer.json with security scripts and suggestions

The integration report documents:
- PHP version incompatibility (php-aegis requires 8.1+, plugin needs 7.4+)
- sanctify-php is a Haskell build tool, not a runtime PHP library
- Recommendations for upstream improvements to both libraries

Version bumped to 1.1.0 to reflect security enhancements.
@hyperpolymath hyperpolymath merged commit 5837f27 into main Dec 27, 2025
0 of 11 checks passed
@hyperpolymath hyperpolymath deleted the claude/integrate-php-libraries-Q6W9F branch December 27, 2025 00:13
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