Skip to content

CAMEL-23123: Fix S3Presigner ignoring forcePathStyle configuration#21690

Merged
Croway merged 1 commit intoapache:mainfrom
Croway:CAMEL-23123
Mar 3, 2026
Merged

CAMEL-23123: Fix S3Presigner ignoring forcePathStyle configuration#21690
Croway merged 1 commit intoapache:mainfrom
Croway:CAMEL-23123

Conversation

@Croway
Copy link
Contributor

@Croway Croway commented Mar 3, 2026

Description

  • Extract duplicated S3Presigner creation logic from createDownloadLink() and createUploadLink() into a shared getOrCreatePresigner() helper method
  • Apply forcePathStyle to the presigner via S3Configuration.pathStyleAccessEnabled(true) when the option is enabled
  • Add integration test verifying path-style vs virtual-hosted-style presigned URL generation

@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2026

🌟 Thank you for your contribution to the Apache Camel project! 🌟
🤖 CI automation will test this PR automatically.

🐫 Apache Camel Committers, please review the following items:

  • First-time contributors require MANUAL approval for the GitHub Actions to run
  • You can use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot although they are normally detected and executed by CI.
  • You can label PRs using build-all, build-dependents, skip-tests and test-dependents to fine-tune the checks executed by this PR.
  • Build and test logs are available in the summary page. Only Apache Camel committers have access to the summary.

⚠️ Be careful when sharing logs. Review their contents before sharing them publicly.

Co-Authored-By: Bruno Meseguer <brunoNetId@users.noreply.github.com>
@Croway Croway merged commit befe832 into apache:main Mar 3, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants