Skip to content

Conversation

kelvinkipruto
Copy link
Contributor

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

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Screenshots

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation

Also configure S3 plugin with maxAttempts and signedDownloads options
Add ft/upgrade-payload branch to trigger dev deployment for testing purposes
@kilemensi
Copy link
Member

kilemensi commented Oct 17, 2025

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 maxSocket for now (something close to 1000 but should be an env var).

@kilemensi
Copy link
Member

... 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",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sort these alphabetically

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.

3 participants