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

Contentful Image API returns error if requesting images with dimensions above 4000px #3409

Closed
baseten opened this issue Jan 4, 2018 · 4 comments
Labels
status: blocked This issue/PR can't be solved at the moment and shouldn't be closed/merged

Comments

@baseten
Copy link

baseten commented Jan 4, 2018

The responsiveSizes, responsiveResolution and resize graphQL fields from gatsby-source-contentful can request images from the contentful API which are over 4000px in size. These will return errors from contentful.

@calcsam
Copy link
Contributor

calcsam commented Jan 4, 2018

Have you filed this with Contentful and gotten any feedback from them? Do you have a repro repo (even a bash script CURL-ing a resource would be great).

If the issue is with an underlying CMS's API, we should get context from them before patching our plugin.

@baseten
Copy link
Author

baseten commented Jan 4, 2018

No, I've not filed with Contentful, but will get in touch with them now. You can see the issue easily if you open a contentful image in a browser tab. Using the below image from gatsby's contentful example site:

https://images.contentful.com/rocybtov1ozk/KTRF62Q4gg60q6WCsWKw8/a8b2e93ac83fbbbb7bf9fba9f92b018e/soso.clock.jpg?w=4000&h=4000&q=50&fit=fill

If you change either the w or h to 4001 you'll receive an error saying valid transformation: False

@fk fk added API/Plugins status: blocked This issue/PR can't be solved at the moment and shouldn't be closed/merged labels Jan 19, 2018
@robinpyon
Copy link
Contributor

Came across this issue too. Reached out to Contentful who confirmed the 4000px width/height limit is real and documented, though easy to miss.

@LekoArts
Copy link
Contributor

LekoArts commented Dec 4, 2018

Closing this as the original question was answered. If you want to document this limitation, we'd be happy to have a PR from you. It would probably be best to link to contentful's documentation so we don't have to be in sync with theirs.

@LekoArts LekoArts closed this as completed Dec 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: blocked This issue/PR can't be solved at the moment and shouldn't be closed/merged
Projects
None yet
Development

No branches or pull requests

6 participants