search

madmatt / silverstripe-encrypt-at-rest

Encrypt data at rest in your database
7 stars 11 forks source link

Additional PHP8.1 hardening #31

Closed scott-nz closed 1 year ago

scott-nz commented 1 year ago

Uses the same logic as already included in EncryptedText.

NOTE: This is required for Silverstripe 4 and 5 so should be merged in before https://github.com/madmatt/silverstripe-encrypt-at-rest/pull/30

Firesphere commented 1 year ago

@madmatt I'm pretty happy with the solution this way, and it reduces code smell. Any final thoughts from you¿

madmatt commented 1 year ago

Looks good to me, thanks @scott-nz and @Firesphere!