-
Notifications
You must be signed in to change notification settings - Fork 44
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
Cannot specify output format for path without extension #22
Comments
This would be fixed by using a querystring protocol for modifiers, instead of parsing the urls. |
This is related to tripviss#22
Personally, I'd love to use the query string too (it's the proper On Thu, 6 Oct 2016, 00:59 Pedro Tacla Yamada, [email protected]
|
It works fine with cloudfront afaik. There's an initial lousy impl. in my Em qua, 5 de out de 2016 14:05, Teoh Han Hui [email protected]
|
Just to back this up a bit, I've seen other resizer CDNs, like filpicker's CDN and imgix.com do this; https://docs.imgix.com/setup/serving-images I sort of doubt that the querystring isn't taken into account on all but only very few CDNs. Because the querystring is part of the URI, it's part of the key you really want to cache. CDNs that don't respect it seem broken to me. |
e.g. https://lh4.googleusercontent.com/-P4oxiHniRcc/V1Q0kGaDVMI/AAAAAAAAS44/JV4M46DxtWc54Rjb1pv2i29CACoUmS_6QCLIB/w4000-h3000-k/
There is no way to specify the output format since the current parsing requires the input format extension before it (e.g.
.jpg.webp
).The text was updated successfully, but these errors were encountered: