Skip to content

refactor: cleanup Security#9972

Merged
paulbalandan merged 4 commits intocodeigniter4:4.8from
paulbalandan:cleanup-security
Feb 21, 2026
Merged

refactor: cleanup Security#9972
paulbalandan merged 4 commits intocodeigniter4:4.8from
paulbalandan:cleanup-security

Conversation

@paulbalandan
Copy link
Member

@paulbalandan paulbalandan commented Feb 21, 2026

Description

  • Removes deprecated properties and methods
  • Reorder methods (for visual clarity)
  • Add static to verify() method (potentially breaking)
  • Refactor Security
    • Faster and safer derandomize()
    • Early returns to reduce nested ifs
    • No unneeded persisting cookie instance (as used only once)
    • No unneeded persisting request instance (as used only once)

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value (without duplication)
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@paulbalandan paulbalandan added refactor Pull requests that refactor code breaking change Pull requests that may break existing functionalities 4.8 PRs that target the `4.8` branch. labels Feb 21, 2026
@paulbalandan paulbalandan merged commit e3e85b4 into codeigniter4:4.8 Feb 21, 2026
50 checks passed
@paulbalandan paulbalandan deleted the cleanup-security branch February 21, 2026 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4.8 PRs that target the `4.8` branch. breaking change Pull requests that may break existing functionalities refactor Pull requests that refactor code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants