Re: Status of pull request

From: 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

« previous php.internals (#65896) next »