Also webp should be supported by getimagesize, getimagesizefromstring, image_type_to_extension and image_type_to_mime_type.

+1 for WEBP support in getimagesize(), please!

WEBP is becoming increasingly popular, and getimagesize() is so handy to get image information without having to load the whole file using Imagick, for example.

I just spent an hour trying to figure out why webp isn't working in getimagesize only to eventually arrive at this bug. Please add support for it.

Changing the category here to be "GetImageSize" as internally exif_imagetype() re-uses an internal function we use for GetImageSize(), and exposes it to userland, despite not having anything exif related code in it.

This bug report is over 3 years old and webp is gaining traction (http://caniuse.com/#search=webp). Is there any progress on this issue?

Implemented with <http://git.php.net/?p=php-src.git;a=commit;h=14d4ee93>.

Documented with <http://svn.php.net/viewvc?view=revision&revision=340375>.

Why ignore lossless webp? Can you open a new issue about that with an explanation?

Also what exactly does it mean that only lossy webp is supported? Does it mean that if I call getimagesize on a lossless webp it won't return the needed info?

> Does it mean that if I call getimagesize on a lossless webp it
> won't return the needed info?

Indeed, getimagesize() would return FALSE in this case.

Actually, I thought that lossless WebP isn't very common and that
it wouldn't be supported by libgd, but at least the latter
assumption was wrong – imagecreatefromwebp() is able to read
lossless WebP (at least the simple format without alpha; haven't
yet checked further). So it makes sense to add support for simple
lossless WebP (VP8L) to getimagesize() and friends.

VP8L support implemented with <http://git.php.net/?p=php-src.git;a=commit;h=59cd8eb7>.

Very nice! :-)

Now is there a reason not to implement VP8X as well? It seems that the "Canvas Width Minus One" and "Canvas Height Minus One" are always at the same position so it should be easy. https://developers.google.com/speed/webp/docs/riff_container#extended_file_format

Of course it depends on whether imagecreatefromwebp supports VP8X or not.

> Of course it depends on whether imagecreatefromwebp supports
> VP8X or not.

Not necessarily; for instance, getimagesize() supports BMP, but
GD still doesn't.

Anyhow, as I've just learned imagecreatefromwebp() does support
VP8X (even stored color profiles are applied), so it makes indeed
sense to add support for VP8X to getimagesize().

VP8X support added with commit <http://git.php.net/?p=php-src.git;a=commit;h=1d977e55>.

Is there any chance this patch would be accepted into the 5.6 branch?

Not all of us are able to move directly to 7.1 to take advantage of this, and other webp supporting functions like imagecreatewebp are in 5.5+

> Is there any chance this patch would be accepted into the 5.6
> branch?

This ticket had been rightly filed as feature request, and we
usually don't add new features to stable branches for QA reasons.
Note also that active support for PHP 5.6 ends in about 6 weeks,
so it's rather unlikely that this patch will be backported to 5.6.

If you really have to stick with PHP 5.6 for now, consider a
userland implementation. If you only have to detect whether a file
is in WebP format, it should be sufficient to check whether the
first three bytes of the file are `VP8`.