Skip to content

Conversation

@CarlSchwan
Copy link
Member

@CarlSchwan CarlSchwan commented Jul 30, 2021

This is the case on rhel7 there PHP is not compiled with argon2 support and php-sodium is a weak dependency and not installed by default. On other distributions and rhel8, php-sodium or argon2 is already installed so there is less risk of it breaking.

Related to #21100

Signed-off-by: Carl Schwan [email protected]

@CarlSchwan CarlSchwan force-pushed the work/carl/sodium-check branch from f9587da to 8e51a5f Compare July 30, 2021 13:12
Copy link
Contributor

@kesselb kesselb left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@kesselb kesselb added 3. to review Waiting for reviews enhancement labels Jul 30, 2021
@ChristophWurst ChristophWurst merged commit 43e41ca into master Jul 30, 2021
@ChristophWurst ChristophWurst deleted the work/carl/sodium-check branch July 30, 2021 15:34
@ChristophWurst
Copy link
Member

backport?

@CarlSchwan
Copy link
Member Author

/backport to stable22

@CarlSchwan
Copy link
Member Author

/backport to stable21

@CarlSchwan
Copy link
Member Author

/backport to stable20

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants