search

hanneskod / libmergepdf

PHP library for merging multiple PDFs
390 stars 77 forks source link

php version issue? #20

Closed severn-leon closed 7 years ago

severn-leon commented 7 years ago

Hi,

I'm using your package as part of a laravel application and it works OK on our staging server which is php 5.6, unbuntu 14.04.

Have now moved it to a different server which runs php 7.1, unbuntu 16.04 and the app falls over as soon as I hit this line : $m = new \iio\libmergepdf\Merger(); It causes an nginx 502 error, so I can't see any debug information. If I comment that line out everything works fine.

Any help you can give me working out what's going wrong would be much appreciated.

Thanks. Leon

severn-leon commented 7 years ago

This is from the nginx error log :

2017/01/12 23:46:14 [error] 9602#9602: *1624 recv() failed (104: Connection reset by peer) while reading response header from upstream, client: ip-address, server: domain-name, request: "GET /manage/Meeting/504/board-pack HTTP/2.0", upstream: "fastcgi://unix:/var/run/php/php7.1-fpm.sock:", host: "host-name"

hanneskod commented 7 years ago

Hi and thanks for reporting!

This seems to be related to php7.1, opcache and fpdi (which libmergepdf uses internally). Please see https://github.com/Setasign/FPDI/issues/13. Other than that I don't really have an answer at the moment..

Hannes

severn-leon commented 7 years ago

Thanks very much Hannes, that looks really useful, can't work on it today, but will let you know how I get on as soon as I get some time to find a fix.

Cheers! Leon

On Fri, Jan 13, 2017 at 8:32 AM, Hannes Forsgård notifications@github.com wrote:

Hi and thanks for reporting!

This seems to be related to php7.1, opcache and fpdi (which libmergepdf uses internally). Please see Setasign/FPDI#13 https://github.com/Setasign/FPDI/issues/13. Other than that I don't really have an answer at the moment..

Hannes

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/hanneskod/libmergepdf/issues/20#issuecomment-272388383, or mute the thread https://github.com/notifications/unsubscribe-auth/AESCqLV2RZnGMPii110cORRvKT0JK8lMks5rRzaDgaJpZM4LiXjP .

-- Leon Cottell Severn Solutions Ltd.

mobile: 0797 653 0543 email: leon@severn-solutions.co.uk web: www.severn-solutions.co.uk

hanneskod commented 7 years ago

👍

severn-leon commented 7 years ago

Changed PHP version to 7.0 and working OK now.

Thanks.

JanSlabon commented 7 years ago

PHP 7.1.1 is out - it should fix this issue.

hanneskod commented 7 years ago

Closing as this is not libmergepdf per se..