-
Notifications
You must be signed in to change notification settings - Fork 2
build(payload): fix images not loading #1334
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
base: main
Are you sure you want to change the base?
Conversation
Also configure S3 plugin with maxAttempts and signedDownloads options
Add ft/upgrade-payload branch to trigger dev deployment for testing purposes
For our case (or at least for today), if the upgrade doesn't fix this issue then this looks like a promising fix i.e. increase |
... and again, I believe building Docker locally on M3 Pro should be faster than on a GitHub Action so you should be able to test and iterate much faster that way. I believe you have values for all of these envs/secrets (and they should also be in the vault). |
Add S3_MAX_ATTEMPTS, S3_MAX_SOCKETS and S3_CONNECTION_TIMEOUT env variables to configure S3 client connection settings. This improves reliability and performance of S3 operations. ci: remove temporary testing branch from workflow refactor(incubator): add title prop and Typography component
"S3_SECRET_ACCESS_KEY", | ||
"S3_MAX_ATTEMPTS", | ||
"S3_MAX_SOCKETS", | ||
"S3_CONNECTION_TIMEOUT", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sort these alphabetically
Description
This PR attempts to address an issue where images intermittently load.
Similar issues
payloadcms/payload#6382
payloadcms/payload#12366
A few individuals facing the issue have resolved it by updating to the latest version and serving the files directly from S3 by using the
signedDownloads: true
Type of change
Screenshots
Checklist: