Re: [PHP-DEV] [RFC] [VOTE] Improve openssl_random_pseudo_bytes()

2018-11-03 Thread Niklas Keller
Hey Sammy, could you please turn your patch into a PR, so it's easier to comment? Thanks, Niklas Am Fr., 2. Nov. 2018 um 20:30 Uhr schrieb Sammy Kaye Powers : > > Hey internals! > > Given there has been at least two weeks of discussion around the RFC, > "Improve openssl_random_pseudo_bytes()" and

Re: [PHP-DEV] [RFC] [VOTE] Improve openssl_random_pseudo_bytes()

2018-11-03 Thread Nikita Popov
On Fri, Nov 2, 2018 at 8:30 PM Sammy Kaye Powers wrote: > Hey internals! > > Given there has been at least two weeks of discussion around the RFC, > "Improve openssl_random_pseudo_bytes()" and there doesn't seem to be > any outstanding issues, I have moved the RFC into voting phase. > > The votin

Re: [PHP-DEV] [RFC] [VOTE] Improve openssl_random_pseudo_bytes()

2018-11-03 Thread Sammy Kaye Powers
@Niklas: Ooops! Thanks for pointing that out. Done! :) https://github.com/php/php-src/pull/3649 @Nikita > IMHO we should just document that it is guaranteed to always be true if the > call succeeds and checking it is not necessary. But we should not penalize > existing code that is checking the