Skip to content

chore: upgrade temporal SDK to v1.41.0 and envconfig to v1.0.0#975

Merged
chaptersix merged 2 commits intotemporalio:mainfrom
chaptersix:fix-tls-sdk
Apr 8, 2026
Merged

chore: upgrade temporal SDK to v1.41.0 and envconfig to v1.0.0#975
chaptersix merged 2 commits intotemporalio:mainfrom
chaptersix:fix-tls-sdk

Conversation

@chaptersix
Copy link
Copy Markdown
Contributor

@chaptersix chaptersix commented Apr 6, 2026

Summary

  • Upgrades go.temporal.io/sdk from v1.38.0 to v1.41.0 in both root and cliext modules
  • Upgrades go.temporal.io/sdk/contrib/envconfig from v0.1.0 to v1.0.0 in both modules

Why

envconfig v0.1.0 had a bug where ToClientOptions did not set ConnectionOptions.TLSDisabled = true when a profile had TLS disabled. This meant that passing --api-key X --tls=false would still dial with TLS enabled (because the SDK auto-enables TLS when credentials are present), overriding the user's explicit --tls=false flag.

Fixed upstream in temporalio/sdk-go#2205, released in envconfig v1.0.0.

- Fixes TLS disabled handling: envconfig v1.0.0 now sets
  ConnectionOptions.TLSDisabled = true when profile TLS is disabled,
  resolving the bug where --api-key X --tls=false would still dial
  with TLS enabled
@chaptersix chaptersix marked this pull request as ready for review April 6, 2026 18:51
@chaptersix chaptersix requested a review from a team as a code owner April 6, 2026 18:51
@chaptersix chaptersix requested a review from dandavison April 6, 2026 19:03
Copy link
Copy Markdown

@jaypipes jaypipes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes please :)

Copy link
Copy Markdown
Contributor

@dandavison dandavison left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, LGTM!

@chaptersix chaptersix merged commit 2da907c into temporalio:main Apr 8, 2026
9 checks passed
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.

3 participants