Recommendation: require RandomLib if you use it as a fallback

zendframework / zend-math

Math component from Zend Framework

BSD 3-Clause "New" or "Revised" License

54 stars 25 forks source link

Recommendation: require RandomLib if you use it as a fallback #17

Closed Saeven closed 8 years ago

Saeven commented 8 years ago

Ran into this: https://github.com/zendframework/zend-math/blob/master/src/Rand.php#L75

I think userland (at least me) would rather have composer install this rather than run into this message.

johnnymallie commented 8 years ago

Hi ! I had the same problem this morning after a composer update.

I agree with Saeven, even tough I'm not sure if it's the best fit.

Looking forward for a solution or a guideline to follow

ezimuel commented 8 years ago

@Saeven we put this message to alert about the need of ircmaxell/random-lib if Mcrypt is not installed. We removed the usage of OpenSSL for security reason, see ZF2016-01. You can easily fix it adding the library in composer, using the following command:

composer require ircmaxell/random-lib:1.*

In composer.json we added this suggestion long time ago.