-
Notifications
You must be signed in to change notification settings - Fork 730
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
Update the data type of OtlpExporter option Endpoint to string #1786
Update the data type of OtlpExporter option Endpoint to string #1786
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1786 +/- ##
=======================================
Coverage 83.89% 83.89%
=======================================
Files 187 187
Lines 5956 5956
=======================================
Hits 4997 4997
Misses 959 959
|
@@ -32,9 +32,9 @@ public class OtlpExporterOptions | |||
#if NETSTANDARD2_1 | |||
/// <summary> | |||
/// Gets or sets the target to which the exporter is going to send traces or metrics. | |||
/// The valid syntax is described at https://github.com/grpc/grpc/blob/master/doc/naming.md. | |||
/// The valid syntax is described at https://github.com/grpc/grpc/blob/master/doc/naming.md. The endpoint value should start with http or https. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The link to valid syntax is not really applicable for NET2_1, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that is correct. That naming is for gRPC Core only I think.
Closing this PR as this was fixed in #1789 |
Partially Fixes #1778
Changes
CHANGELOG.md
updated for non-trivial changes