Skip to content

Add mTLS support for remote write#186

Merged
saswatamcode merged 1 commit into
prometheus-community:mainfrom
CharlieTLe:worktree-rosy-napping-thacker
Jul 2, 2026
Merged

Add mTLS support for remote write#186
saswatamcode merged 1 commit into
prometheus-community:mainfrom
CharlieTLe:worktree-rosy-napping-thacker

Conversation

@CharlieTLe

Copy link
Copy Markdown
Contributor

Adds --tls-client-cert-file, --tls-client-key-file, and --tls-ca-cert-file flags so avalanche can authenticate to remote-write endpoints that require mTLS or a custom CA. ConfigWrite.TLSClientConfig now holds *tls.Config to avoid copying the embedded sync.RWMutex. A warning is logged when --tls-client-insecure is combined with --tls-ca-cert-file since the CA is silently bypassed in that case.

Includes tests covering each flag combination and README documentation.

@CharlieTLe CharlieTLe force-pushed the worktree-rosy-napping-thacker branch from 75e9baa to 61428cb Compare July 1, 2026 20:22
Adds --tls-client-cert-file, --tls-client-key-file, and --tls-ca-cert-file
flags so avalanche can authenticate to remote-write endpoints that require
mTLS or a custom CA. ConfigWrite.TLSClientConfig now holds *tls.Config to
avoid copying the embedded sync.RWMutex. A warning is logged when
--tls-client-insecure is combined with --tls-ca-cert-file since the CA is
silently bypassed in that case.

Includes tests covering each flag combination and README documentation.

Signed-off-by: Charlie Le <charlie_le@apple.com>
@CharlieTLe CharlieTLe force-pushed the worktree-rosy-napping-thacker branch from 61428cb to 7bfd24d Compare July 1, 2026 20:23

@saswatamcode saswatamcode left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks! LGTM!

@saswatamcode saswatamcode merged commit 79a1b45 into prometheus-community:main Jul 2, 2026
11 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.

2 participants