Skip to content

Migrate to AWS SDK v2, remove build tag requirements, always-on logging#238

Open
osmin-lazo wants to merge 1 commit into
aws-cloudformation:masterfrom
osmin-lazo:migrate-sdk-v2-always-on-logging
Open

Migrate to AWS SDK v2, remove build tag requirements, always-on logging#238
osmin-lazo wants to merge 1 commit into
aws-cloudformation:masterfrom
osmin-lazo:migrate-sdk-v2-always-on-logging

Conversation

@osmin-lazo

@osmin-lazo osmin-lazo commented Jun 17, 2026

Copy link
Copy Markdown
  • Migrate all packages from aws-sdk-go v1 to aws-sdk-go-v2
  • Remove -tags logging/callback/scheduler build tag requirements
  • Logging is now always enabled (deleted *_notag.go files)
  • Replace session.Session with aws.Config (handler.Request.Config)
  • Replace cloudwatchiface/cloudformationiface with explicit interfaces
  • Fall back to stdout when ProviderLogGroupName is empty
  • Update Go minimum version to 1.21

Breaking changes:

  • handler.Request.Session (*session.Session) -> handler.Request.Config (*aws.Config)
  • credentials.SessionFromCredentialsProvider -> credentials.ConfigFromCredentialsProvider
  • No longer requires -tags logging to enable CloudWatch logging

Fixes #187, #224, #237

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

- Migrate all packages from aws-sdk-go v1 to aws-sdk-go-v2
- Remove -tags logging/callback/scheduler build tag requirements
- Logging is now always enabled (deleted *_notag.go files)
- Replace session.Session with aws.Config (handler.Request.Config)
- Replace cloudwatchiface/cloudformationiface with explicit interfaces
- Fall back to stdout when ProviderLogGroupName is empty
- Update Go minimum version to 1.21

Breaking changes:
- handler.Request.Session (*session.Session) -> handler.Request.Config (*aws.Config)
- credentials.SessionFromCredentialsProvider -> credentials.ConfigFromCredentialsProvider
- No longer requires -tags logging to enable CloudWatch logging

Fixes aws-cloudformation#187
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.

Logging unavailable at runtime unless logging tag passed to compile

1 participant