Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update sodium_compat to 1.20.0. #4400

Closed

Conversation

costdev
Copy link
Contributor

@costdev costdev commented Apr 30, 2023

The latest version of sodium_compat includes a fix for the Poly1305 implementation.

Release Notes:
https://github.com/paragonie/sodium_compat/releases/tag/v1.20.0

A full list of changes in this update can be found on GitHub:
paragonie/sodium_compat@v1.19.0...v1.20.0

Trac ticket: https://core.trac.wordpress.org/ticket/58224

@@ -1360,6 +1360,8 @@ function sodium_memcmp($string1, $string2)
* @return void
* @throws SodiumException
* @throws TypeError
*
* @psalm-suppress ReferenceConstraintViolation
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While this isn't strictly necessary, it seemed like this change was worth adding to minimize differences with the original source.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏻 I agree with this reasoning.

@costdev costdev marked this pull request as ready for review April 30, 2023 03:31
Copy link
Member

@mukeshpanchal27 mukeshpanchal27 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @costdev, PR look good to me.

Copy link
Member

@jrfnl jrfnl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM

@SergeyBiryukov
Copy link
Member

Thanks for the PR! Merged in r55699.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants