#64079 closed task (blessed) (fixed)
Update Sodium Compat to 1.23.0 for better performance
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 6.9 | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | External Libraries | Keywords: | needs-patch |
| Focuses: | performance | Cc: |
Description
Hi WordPress team. I've been busy since #64008 working to improve performance and test coverage
- https://github.com/paragonie/sodium_compat/releases/tag/v1.22.0
- https://github.com/paragonie/sodium_compat/releases/tag/v1.23.0
- https://github.com/paragonie/sodium_compat/compare/v1.21.2...v1.23.0
This change affects a lot of internal classes, but see https://github.com/paragonie/sodium_compat/pull/198 for what this much diff buys you. That's about a 7% to 12% increase for the PHPUnit test suite across PHP versions. The more you touch curve25519, without ext-sodium installed, the more pronounced this performance benefit will be.
Change History (6)
#2
@
5 weeks ago
- Keywords needs-patch added
- Milestone changed from Awaiting Review to 6.9
Thanks for the ticket @paragoninitiativeenterprises 👍
#4
@
5 weeks ago
- Owner set to SergeyBiryukov
- Resolution set to fixed
- Status changed from new to closed
In 60905:
#5
@
5 weeks ago
@SergeyBiryukov This sort of change really needs to go in via a PR first instead of being committed directly without even a patch file. The tests didn't have a chance to run prior to commit. Nowhere is it documented how the changes in this commit were generated.
#6
@
5 weeks ago
@johnbillion You raise a good point. Happy to attest to the changeset this time:
Scrolling through https://core.trac.wordpress.org/changeset/60905, it appears to be the diff from https://github.com/paragonie/sodium_compat/compare/v1.21.2...v1.23.0 without the changes to our tests/ directory. I don't observe any errant additions or exclusions beyond the omission of tests (which is normal for a sodium_compat update).
If anything does break from this update, it would be astonishing to me and should be investigated as a sodium_compat bug rather than a patch import issue.
Sorry, forgot to highlight something from v1.22.0:
I doubt many WordPress sites or plugins used this feature to begin with, but I wanted to make sure it was flagged.