Skip to content

Commit

Permalink
Revert "Update default OTLP http port (open-telemetry#939)"
Browse files Browse the repository at this point in the history
This reverts commit 05444a8.
  • Loading branch information
ThomsonTan committed Aug 5, 2021
1 parent 05444a8 commit 5df066c
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 5 deletions.
7 changes: 3 additions & 4 deletions exporters/otlp/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -44,10 +44,9 @@ auto exporter = std::unique_ptr<sdktrace::SpanExporter>(new otlp::OtlpHttpExport

### Configuration options

| Option | Default |
| ------------------------------------ |----------------- |
| `OtlpGrpcExporterOptions.endpoint` | `localhost:4317` |
| `OtlpHttpExporterOptions.endpoint` | `localhost:4318` |
| Option | Default |
| ------------ |----------------- |
| `endpoint` | `localhost:4317` |

## Example

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ struct OtlpHttpExporterOptions
// @see
// https:/open-telemetry/opentelemetry-specification/blob/main/specification/protocol/otlp.md
// @see https:/open-telemetry/opentelemetry-collector/tree/main/receiver/otlpreceiver
std::string url = std::string("http://localhost:4318") + kDefaultTracePath;
std::string url = std::string("http://localhost:4317") + kDefaultTracePath;

// By default, post json data
HttpRequestContentType content_type = HttpRequestContentType::kJson;
Expand Down

0 comments on commit 5df066c

Please sign in to comment.