Req #65187 [Csd]: exif_read_data : add support for stream resource

From: Date: Wed, 12 Jul 2017 03:48:50 +0000
Subject: Req #65187 [Csd]: exif_read_data : add support for stream resource
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-210003@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=65187&edit=1 ID: 65187 Updated by: kalle@php.net Reported by: imprec at gmail dot com Summary: exif_read_data : add support for stream resource Status: Closed Type: Feature/Change Request Package: EXIF related PHP Version: Irrelevant Assigned To: kalle Block user comment: N Private report: N New Comment: ps, it also works for exif_thumbnail() Previous Comments: ------------------------------------------------------------------------ [2017-07-12 03:45:02] kalle@php.net I have just implemented this in master, it will be available as of PHP 7.2.0 Beta 1! Thanks for helping PHP become EVEN greater! ------------------------------------------------------------------------ [2016-08-05 04:08:39] kalle@php.net I'm currently working on a patch, and a side thing that is coming out of this is an additional $context parameter for use with non streams, since internally PHP is creating a stream with no context, this can help work around supplying direct addresses like locations thats gated by custom protocols or HTTPS which would need peer verification or similar. I expect this patch to only go in PHP 7.2 and onwards. ------------------------------------------------------------------------ [2016-08-03 10:04:12] kalle@php.net It seems like all the exif_* functions do not support streams, I'm gonna take a look at it ------------------------------------------------------------------------ [2013-11-25 13:13:45] imprec at gmail dot com After reading the comment of maggus dot staab at googlemail dot com, I reopen the ticket ------------------------------------------------------------------------ [2013-11-25 13:10:34] maggus dot staab at googlemail dot com while the posted example will work, it requires a lot memory on php-side because you need to read the whole file + you need additional space for the base64 encoded string. so its far away from an efficient solution and will not work for big/huge images. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at https://bugs.php.net/bug.php?id=65187 -- Edit this bug report at https://bugs.php.net/bug.php?id=65187&edit=1

« previous php.bugs (#210003) next »