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
[configtls] Rename configuration structs to end in Config
and remove TLS
prefix
#9474
Closed
3 tasks done
Labels
Milestone
Comments
mx-psi
added
release:required-for-ga
Must be resolved before GA release
area:config
labels
Feb 6, 2024
Few others:
|
arjunmahishi
added a commit
to arjunmahishi/opentelemetry-collector
that referenced
this issue
Feb 6, 2024
This commit renames the following structs: * TLSClientSetting to ClientConfig * TLSServerSetting to ServerConfig * TLSSetting to Config This is based on the naming convention followed in other config packages. Fixes open-telemetry#9474
arjunmahishi
added a commit
to arjunmahishi/opentelemetry-collector
that referenced
this issue
Feb 7, 2024
This commit renames the following structs: * TLSClientSetting to ClientConfig * TLSServerSetting to ServerConfig * TLSSetting to Config This is based on the naming convention followed in other config packages. Fixes open-telemetry#9474
arjunmahishi
added a commit
to arjunmahishi/opentelemetry-collector
that referenced
this issue
Feb 7, 2024
This commit renames the following structs: * TLSClientSetting to ClientConfig * TLSServerSetting to ServerConfig * TLSSetting to Config This is based on the naming convention followed in other config packages. Fixes open-telemetry#9474
arjunmahishi
added a commit
to arjunmahishi/opentelemetry-collector
that referenced
this issue
Feb 28, 2024
This commit renames the following structs: * TLSClientSetting to ClientConfig * TLSServerSetting to ServerConfig * TLSSetting to Config This is based on the naming convention followed in other config packages. Fixes open-telemetry#9474
dmitryax
pushed a commit
that referenced
this issue
Feb 29, 2024
**Description:** Simply renames a few structs in the `configtls` package for consistence. `TLSClientSetting` to `ClientConfig` `TLSServerSetting` to `ServerConfig` `TLSSetting` to `Config` **Link to tracking Issue:** Fixes #9474
mx-psi
pushed a commit
that referenced
this issue
Mar 27, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
To be consistent with the recent changes in
configgrpc
andconfighttp
we should:configtls.TLSClientSetting
toconfigtls.ClientConfig
configtls.TLSServerSetting
toconfigtls.ServerConfig
configtls.TLSSetting
toconfigtls.Config
The text was updated successfully, but these errors were encountered: