Skip to content

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented May 7, 2025

Proposed commit message

Use AWS-specific HTTP client awshttp.BuildableClient with overrides matching generic HTTP client of the processor. This allows the use of custom CA bundle loading and avoids failing internal to AWS SDK.

Fix was tested in an EC2 environment with AWS_CA_BUNDLE and it works as expected.

image

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

Related issues

Fixes #44186


This is an automatic backport of pull request #44189 done by Mergify.

* use aws http client

Signed-off-by: Kavindu Dodanduwa <kavindu.dodanduwa@elastic.co>

* add changelog entry

Signed-off-by: Kavindu Dodanduwa <kavindu.dodanduwa@elastic.co>

* review change : use chaining

Signed-off-by: Kavindu Dodanduwa <kavindu.dodanduwa@elastic.co>

---------

Signed-off-by: Kavindu Dodanduwa <kavindu.dodanduwa@elastic.co>
(cherry picked from commit f54c496)
@mergify mergify bot requested a review from a team as a code owner May 7, 2025 15:51
@mergify mergify bot added the backport label May 7, 2025
@mergify mergify bot added the backport label May 7, 2025
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label May 7, 2025
@github-actions github-actions bot added Team:obs-ds-hosted-services Label for the Observability Hosted Services team bugfix labels May 7, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ds-hosted-services (Team:obs-ds-hosted-services)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label May 7, 2025
@Kavindu-Dodan Kavindu-Dodan merged commit 4e1097f into 9.0 May 7, 2025
172 checks passed
@Kavindu-Dodan Kavindu-Dodan deleted the mergify/bp/9.0/pr-44189 branch May 7, 2025 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport bugfix Team:obs-ds-hosted-services Label for the Observability Hosted Services team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants