Req #65187 [Csd]: exif_read_data : add support for stream resource
| From: | kalle@php.net | 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