Re: Status of pull request
| From: | Stas Malyshev | Date: | Sun, 17 Feb 2013 10:12:24 +0000 |
| Subject: | Re: Status of pull request | ||
| References: | 1 | Groups: | php.internals |
| Request: | Send a blank email to internals+get-65896@lists.php.net to get a copy of this message | ||
Hi!
> The pull request addresses bug fix/feature request #38917 implementing
> native signed public key & challenge support to the OpenSSL extension.
> Details can be found @ https://github.com/php/php-src/pull/267
I've looked into it and two things are missing:
1. Failure scenario tests. See my comments on the patch, in many
functions initializations are missing and error scenarios segfault.
Please fix it.
2. For some reason the diff includes pbkdf2 which is not part of the
patch. No idea why. Could you clean it up?
If this is done, we could get it into 5.5 probably.
Thanks,
--
Stanislav Malyshev, Software Architect
SugarCRM: http://www.sugarcrm.com/
(408)454-6900 ext. 227