Skip to content

Commit

Permalink
bug #32170 [Security/Core] Don't use ParagonIE_Sodium_Compat (nicolas…
Browse files Browse the repository at this point in the history
…-grekas)

This PR was merged into the 3.4 branch.

Discussion
----------

[Security/Core] Don't use ParagonIE_Sodium_Compat

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

This is dead code:
https://github.com/paragonie/sodium_compat#features-excluded-from-this-polyfill

Commits
-------

61ea53d [Security/Core] Don't use ParagonIE_Sodium_Compat
  • Loading branch information
fabpot committed Jun 25, 2019
2 parents 57d73fc + 61ea53d commit 675d463
Showing 1 changed file with 0 additions and 4 deletions.
Expand Up @@ -26,10 +26,6 @@ public static function isSupported()
return true;
}

if (class_exists('ParagonIE_Sodium_Compat') && method_exists('ParagonIE_Sodium_Compat', 'crypto_pwhash_is_available')) {
return \ParagonIE_Sodium_Compat::crypto_pwhash_is_available();
}

return \function_exists('sodium_crypto_pwhash_str') || \extension_loaded('libsodium');
}

Expand Down

0 comments on commit 675d463

Please sign in to comment.