Skip to content

fix(compose): allow overriding S3 endpoint - #3865

Open
pollychen-lab wants to merge 1 commit into
block:mainfrom
pollychen-lab:agent/compose-s3-endpoint-env
Open

fix(compose): allow overriding S3 endpoint#3865
pollychen-lab wants to merge 1 commit into
block:mainfrom
pollychen-lab:agent/compose-s3-endpoint-env

Conversation

@pollychen-lab

Copy link
Copy Markdown

Summary

  • Honor BUZZ_S3_ENDPOINT from compose .env, defaulting to bundled MinIO.
  • Honor BUZZ_S3_ADDRESSING_STYLE from .env, defaulting to path for bundled MinIO.
  • Document both variables in the compose example and README.

Fixes #3794

Validation

  • git diff --check
  • docker compose --env-file .env config with .env copied from .env.example renders BUZZ_S3_ENDPOINT: http://minio:9000 and BUZZ_S3_ADDRESSING_STYLE: path.
  • BUZZ_S3_ENDPOINT=https://example.r2.cloudflarestorage.com BUZZ_S3_ADDRESSING_STYLE=virtual docker compose --env-file .env config renders the external endpoint and virtual addressing.

@pollychen-lab
pollychen-lab requested a review from a team as a code owner July 31, 2026 06:35
@pollychen-lab
pollychen-lab force-pushed the agent/compose-s3-endpoint-env branch from d84cbbe to 09400a8 Compare July 31, 2026 06:35
@Chessing234

Copy link
Copy Markdown
Contributor

looks right for #3794. tiny nit: call out in the readme that path vs virtual still matters for minio vs real s3 so people dont copy the default blindly

@Chessing234

Copy link
Copy Markdown
Contributor

s3 endpoint override was overdue. worth a one-line note in .env.example so people know the compose hardcode is gone.

Signed-off-by: Polly Labs <pollychen.lab@gmail.com>
@pollychen-lab
pollychen-lab force-pushed the agent/compose-s3-endpoint-env branch from 09400a8 to 8cbbd53 Compare July 31, 2026 12:37
@pollychen-lab

Copy link
Copy Markdown
Author

@Chessing234 thank you for checking

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.

fix(compose): BUZZ_S3_ENDPOINT is hardcoded, so .env cannot point media at external S3

2 participants