Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=885344
Shawn Iwinski <siwin...@redhat.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |siwin...@redhat.com
--- Comment #8 from Shawn Iwinski <siwin...@redhat.com> ---
(In reply to comment #7)
> 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)
Removed bundled cert
> SHOULD:
>
> [!]: Latest version is packaged.
> 3.1.2 is released
Updated to 3.1.2
> [!]: %check is present and all tests pass.
> as tests exists in upstream github, please
> ask upstream to include them in the tarball
Will work with upstream to get tests included in PEAR package.
Spec URL:
http://siwinski.fedorapeople.org/rpmbuild/SPECS/php-guzzle-Guzzle.spec
SRPM URL:
http://siwinski.fedorapeople.org/rpmbuild/SRPMS/php-guzzle-Guzzle-3.1.2-1.fc18.src.rpm
--
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=OH78H1qRkB&a=cc_unsubscribe
_______________________________________________
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
