-
Notifications
You must be signed in to change notification settings - Fork 847
[OTLP] Add mTLS Support for OTLP Exporter #6343
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
sandy2008
wants to merge
39
commits into
open-telemetry:main
Choose a base branch
from
sandy2008:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 12 commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
a363508
feat(): add mtls support
sandy2008 faba6ab
refactor(mtls): extract repeated string literals to constants in Otlp…
sandy2008 3044fc3
feat(): add mtls support
sandy2008 7368c9e
feat(): add mtls configurations for certs
sandy2008 c488394
fix(mtls): include filePath in FileNotFoundException for better diagn…
sandy2008 472205a
fix(mtls): use string.Equals to safely compare thumbprints and avoid …
sandy2008 2d69b7c
fix(): remove redundant comment
sandy2008 960037e
fix(): remove redundant comment
sandy2008 6226ac5
fix(): remove redundant comment
sandy2008 966d076
feat(): add support for client key w/ password
sandy2008 879cd6c
refactor(mtls): extract HttpClient default configuration to separate …
sandy2008 c1405cb
refactor(mtls): eliminate magic strings in OtlpMtlsHttpClientFactory
sandy2008 02b9240
feat(mtls): adjust error msg
sandy2008 6ae5efe
feat(mtls): adjust error msg
sandy2008 eadf9c2
feat(mtls): adjust error msg
sandy2008 2c759c6
chore(mtls): when mTLS file permission validation is skipped on unsup…
sandy2008 87633ba
chore(mtls); Remove preemptive file permission validation from mTLS
sandy2008 af26417
refactor(mtls): optimize mTLS HttpClient creation to reduce allocations
sandy2008 0bd7c5e
fix(mtls): format issue
sandy2008 487d517
fix(mtls): format issue
sandy2008 59cea03
chore(mtls): Make OtlpMtlsOptions sealed
sandy2008 b80cd21
fix(mtls): format issue
sandy2008 6e78148
fix(mtls): format issue
sandy2008 0fca40b
chore(README): table inline
sandy2008 fc25a73
fix(): clean up shipped file
sandy2008 c8a5a55
chore(mtls): fix README doc
sandy2008 464201b
chore(mtls): adjust return mtlsClient to 1-liner
sandy2008 ec0db21
chore(mtls): simplify new OtlpMtlsOptions()
sandy2008 84764e8
refactor(mtls): simplify null check and add comment for ExcludeRoot d…
sandy2008 3ec30f9
fix(mtls): Fix client certificate loading when ClientKeyPath is not p…
sandy2008 f30b6f8
chore(mtls): Improve certificate disposal in OtlpMtlsHttpClientFactory
sandy2008 b948ce1
fix(mtls): Add missing using Xunit statements to mTLS test files
sandy2008 3d75694
fix(mtls): suppress SYSLIB0057 warnings in certificate loading code
sandy2008 4d8b83e
fix(mtls): fix test
sandy2008 3611145
chore(): replace #if NET
sandy2008 3f7a6bb
chore(mtls): remove needless pragma warning restore SYSLIB0057
sandy2008 6135c33
fix(): Remove support for client key password and certificate revocat…
sandy2008 e8f122f
Merge branch 'main' into main
sandy2008 cabc172
Merge branch 'main' into main
sandy2008 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.