-
Notifications
You must be signed in to change notification settings - Fork 410
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
Make WITH_OTLP_HTTP_SSL_PREVIEW
mainstream
#2365
Labels
bug
Something isn't working
triage/accepted
Indicates an issue or PR is ready to be actively worked on.
Comments
github-actions
bot
added
the
needs-triage
Indicates an issue or PR lacks a `triage/foo` label and requires one.
label
Oct 12, 2023
Agree |
Successfully using in production for weeks. |
@kkuehl Thanks for the feedback, this is good to know. When nothing is reported, we are in the dark and left to wonder if a feature is working just fine or if it is not even used. Making this mainstream then. |
marcalff
added
triage/accepted
Indicates an issue or PR is ready to be actively worked on.
and removed
needs-triage
Indicates an issue or PR lacks a `triage/foo` label and requires one.
labels
Oct 18, 2023
marcalff
added a commit
to marcalff/opentelemetry-cpp
that referenced
this issue
Oct 18, 2023
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
bug
Something isn't working
triage/accepted
Indicates an issue or PR is ready to be actively worked on.
To discuss.
HTTP SSL support has been stable for a while.
The preview flag can be removed, and the code be mainstream (without ifdef).
This will decrease the overall complexity, and simplify CI builds.
Likewise for
WITH_OTLP_HTTP_SSL_TLS_PREVIEW
.The text was updated successfully, but these errors were encountered: