Skip to content

feat: add NewClientTLSConfigManager and NewDisabledTLSConfigManager#27106

Merged
gwossum merged 2 commits intomaster-1.xfrom
gw/NewDisabledTLSConfigManager
Jan 9, 2026
Merged

feat: add NewClientTLSConfigManager and NewDisabledTLSConfigManager#27106
gwossum merged 2 commits intomaster-1.xfrom
gw/NewDisabledTLSConfigManager

Conversation

@gwossum
Copy link
Copy Markdown
Member

@gwossum gwossum commented Jan 9, 2026

Add convenience constructors NewClientTLSConfigManager and NewDisabledTLSConfigManager, along with associated tests and refactoring.

@gwossum gwossum self-assigned this Jan 9, 2026
Copy link
Copy Markdown
Contributor

@davidby-influx davidby-influx left a comment

Choose a reason for hiding this comment

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

LGTM

@gwossum gwossum merged commit 17e7050 into master-1.x Jan 9, 2026
9 checks passed
@gwossum gwossum deleted the gw/NewDisabledTLSConfigManager branch January 9, 2026 21:20
gwossum added a commit that referenced this pull request Feb 12, 2026
…27106)

Add convenience constructors NewClientTLSConfigManager and NewDisabledTLSConfigManager, along with associated tests and refactoring.

(cherry picked from commit 17e7050)
gwossum added a commit that referenced this pull request Feb 12, 2026
Add `tlsconfig.TLSConfigManager` for managing TLS configurations, handling certificate reloads, logging certificate expiration warnings, etc.

This is backport of the following master-1.x PRs to the 1.12 branch:
- #27100: feat: add TLSConfigManager for managing TLS configuration (clean cherry-pick)
- #27103: feat: add TLSConfigManager.DialWithDialer method (clean cherry-pick)
- #27106: feat: add NewClientTLSConfigManager and NewDisabledTLSConfigManager  (clean cherry-pick)
- #27120: feat: add TLSConfigManager.UseTLS (clean cherry-pick)
- #27150: fix: Clone *tls.Config returned by TLSConfigManager.TLSConfig (clean cherry-pick)
- #27162: feat: multiple TLS configuration improvements  (almost clean cherry-pick)
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