Skip to content

refactor(go-forwarder): add client package#1134

Merged
ndakkoune merged 1 commit into
nabil.dakkoune/go-forwarderfrom
nabil.dakkoune/go-forwarder-client-package
May 22, 2026
Merged

refactor(go-forwarder): add client package#1134
ndakkoune merged 1 commit into
nabil.dakkoune/go-forwarderfrom
nabil.dakkoune/go-forwarder-client-package

Conversation

@ndakkoune
Copy link
Copy Markdown
Contributor

@ndakkoune ndakkoune commented May 22, 2026

What does this PR do?

Quick refactor to minimize future retry implementation scope by creating a client package

Motivation

Testing Guidelines

Additional Notes

I'll deal with the current state (kms/ssm/secretsmanager in config) when I will refactor at the end of the project.

Types of changes

  • Bug fix
  • New feature
  • Breaking change
  • Misc (docs, refactoring, dependency upgrade, etc.)

Check all that apply

  • This PR's description is comprehensive
  • This PR contains breaking changes that are documented in the description
  • This PR introduces new APIs or parameters that are documented and unlikely to change in the foreseeable future
  • This PR impacts documentation, and it has been updated (or a ticket has been logged)
  • This PR's changes are covered by the automated tests
  • This PR collects user input/sensitive content into Datadog
  • This PR passes the integration tests (ask a Datadog member to run the tests)
  • This PR passes the unit tests
  • This PR passes the installation tests (ask a Datadog member to run the tests)

@github-actions github-actions Bot added the aws label May 22, 2026
@ndakkoune ndakkoune marked this pull request as ready for review May 22, 2026 15:04
@ndakkoune ndakkoune requested a review from a team as a code owner May 22, 2026 15:04
@ndakkoune ndakkoune requested a review from ge0Aja May 22, 2026 15:06
@ndakkoune ndakkoune merged commit 0472f16 into nabil.dakkoune/go-forwarder May 22, 2026
10 checks passed
@ndakkoune ndakkoune deleted the nabil.dakkoune/go-forwarder-client-package branch May 22, 2026 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants