search

willnorris / imageproxy

A caching, resizing image proxy written in Go
Apache License 2.0
3.54k stars 482 forks source link

Add support for progressive JPEGs #77

Open jadsonlourenco opened 7 years ago

jadsonlourenco commented 7 years ago

Good feature to have, did you think to support it?

willnorris commented 7 years ago

Do you know of libraries that have support for it? I see no mention in https://golang.org/pkg/image/jpeg/.

willnorris commented 7 years ago

Looks like decoding progressive jpegs is supported, but no mention of encoding.

willnorris commented 7 years ago

And this post from Nigel doesn't sound encouraging.

Nothing is planned.

IIRC, if you want to tackle this yourself, note that there are two dimensions (S and A) to progressivity. For decoding, look near the "spec calls these values Ss, Se, Ah and Al" comment in scan.go.

so the biggest blocker for this would be a Go library that has support for it. I really don't want to link in any C libraries like imagemagick or whatever, so we'd need a Go solution.

jadsonlourenco commented 7 years ago

Ok, thank you for the answer, closing.

willnorris commented 7 years ago

To be clear, I don't think this needs to be closed. I think it's a fine feature request, it's just one that we likely wouldn't be able to add for a while since it's gated on a jpeg library that supports it. But I wouldn't want to lose track of the request.

schmunk42 commented 5 years ago

Any updates on this? Would be really cool to have it.