[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