[Bug 885344] Review Request: php-guzzle-Guzzle - PHP HTTP client library and framework for building RESTful web service clients
bugzilla at redhat.com
bugzilla at redhat.com
Mon Jan 28 08:27:42 UTC 2013
Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=885344
Remi Collet <fedora at famillecollet.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flags| |fedora-review?
--- Comment #7 from Remi Collet <fedora at famillecollet.com> ---
MUST:
[!]: Package contains no bundled libraries.
provides ...Guzzle/Http/Resources/cacert.pem
should use /etc/pki/tls/cert.pem
(ca-certificates on fedora, need to check on RHEL)
See https://fedoraproject.org/wiki/Features/SharedSystemCertificates
In source code Guzzle/Http/Client.php
$opts[CURLOPT_CAINFO] = __DIR__ . '/Resources/cacert.pem'
This will obviously not work, with current version
(should use data_dir).
(this is not a library, but for security reason, using
a bundled copy seems a very bad idea)
SHOULD:
[!]: Latest version is packaged.
3.1.2 is released
[!]: %check is present and all tests pass.
as tests exists in upstream github, please
ask upstream to include them in the tarball
--
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=xkL6Pw2D2v&a=cc_unsubscribe
More information about the package-review
mailing list