Skip to content

Conversation

@generikjano
Copy link

No description provided.

@jimlambie
Copy link
Contributor

@generikjano thanks for the pull request!

Using the image handler for this will currently fail if any image manipulation parameters are applied to the request URL. For example /image.svg?q=80 will result in a 400 error because the underlying image processor doesn't handle SVG.

I believe in order to make this work in a more generic way, CDN should throw away any supplied parameters if the image type is SVG.

@generikjano
Copy link
Author

@generikjano thanks for the pull request!

Using the image handler for this will currently fail if any image manipulation parameters are applied to the request URL. For example /image.svg?q=80 will result in a 400 error because the underlying image processor doesn't handle SVG.

I believe in order to make this work in a more generic way, CDN should throw away any supplied parameters if the image type is SVG.

Hello @jimlambie,

Thanks for the answer.
Agree, I will work on this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants