Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

x/image/webp: non-Alpha VP8X is not implemented error #25738

Closed
chhatanidinesh opened this issue Jun 5, 2018 · 11 comments
Closed

x/image/webp: non-Alpha VP8X is not implemented error #25738

chhatanidinesh opened this issue Jun 5, 2018 · 11 comments
Labels
FeatureRequest FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Milestone
Unreleased

Comments

@chhatanidinesh
Copy link

chhatanidinesh commented Jun 5, 2018 •
edited

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?

1.6.2

Does this issue reproduce with the latest release?

Yes

What operating system and processor architecture are you using (go env)?

Mac, amd64

What did you do?

Resizing of an image.

If possible, provide a recipe for reproducing the error.
image.Decode(bytes.NewReader(bs))
A complete runnable program is good.
A link on play.golang.org is best.

What did you expect to see?

Image decoding should have worked without error

What did you see instead?

webp: non-Alpha VP8X is not implemented
@gopherbot gopherbot added this to the Unreleased milestone Jun 5, 2018
@FiloSottile FiloSottile changed the title x/image: webp: non-Alpha VP8X is not implemented error x/image/webp: non-Alpha VP8X is not implemented error Jun 5, 2018
@FiloSottile FiloSottile added FeatureRequest NeedsFix The path to resolution is known, but the work has not been done. labels Jun 5, 2018
@FiloSottile
Copy link
Contributor

/cc @nigeltao

@nigeltao
Copy link
Contributor

nigeltao commented Jun 5, 2018

Can you attach an example .webp file that triggers the bug?

@chhatanidinesh
Copy link
Author

chhatanidinesh commented Jun 6, 2018 •
edited

Hi @nigeltao ,
This is the webp image which gives this error
https://lh3.googleusercontent.com/Gd45GzTAsvX-JFkM-mBPYjJesdbTrVSrr6kR-rXELsa0i-uIBBg1-HWmQ_LXkQdnEKE=s180-rw

@pingooStar
Copy link

Same problem here, with https://res-2.cloudinary.com/ssenseweb/image/upload/b_white,c_lpad,g_center,h_960,w_960/c_scale,h_680/f_auto,dpr_1.0/182214F087001_1.jpg

@DarthSim
Copy link

Hi,
Same problem with this image: https://s3.amazonaws.com/sked-customer-uploads/564a433accc7c331440002f9/83324ed7d9f4291d49e9bba969687b66.orig.jpg

@totoleo
Copy link

totoleo commented Mar 6, 2019

+1

@jnvillar
Copy link

Any solution to this problem?

@agnivade
Copy link
Contributor

Nobody is working on it.

Also, please note our policy on "+1" type comments - https://github.com/golang/go/wiki/noplusone.

@theoks
Copy link

theoks commented Aug 15, 2019

Same issue here

@chhatanidinesh
Copy link
Author

https://github.com/chai2010/webp should help

@andrius4669 andrius4669 mentioned this issue Apr 10, 2020
Closed
@narrowizard narrowizard mentioned this issue Aug 20, 2020
Closed
@gopherbot
Copy link

Change https://golang.org/cl/249445 mentions this issue: fix(webp): decode non-alpha vp8x error.

gopherbot pushed a commit to golang/image that referenced this issue Sep 21, 2020
@narrowizard @nigeltao
webp: don't reject VP8X that isn't just VP8 + alpha
3a743ba 
We already support VP8 + alpha, but reject e.g. VP8 + EXIF. After this
commit, we still don't implement VP8 + EXIF (or ANIM, ICCP, etc.), but
we now silently ignore the EXIF chunk instead of rejecting it.

Fixes golang/go#25738, golang/go#38341

Change-Id: I4e9cdb718f0768f34336eab9528b82d2c40a3ee1
GitHub-Last-Rev: a0c2e53
GitHub-Pull-Request: #5
Reviewed-on: https://go-review.googlesource.com/c/image/+/249445
Trust: David Symonds <dsymonds@golang.org>
Reviewed-by: Nigel Tao <nigeltao@golang.org>
@golang golang locked and limited conversation to collaborators Dec 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FeatureRequest FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Projects
None yet
Milestone
Unreleased
Development

Successfully merging a pull request may close this issue.

webp: don't reject VP8X that isn't just VP8 + alpha narrowizard/image
10 participants
@DarthSim @FiloSottile @agnivade @totoleo @jnvillar @nigeltao @gopherbot @theoks @chhatanidinesh @pingooStar