Merge pull request #158 from nblumhardt/endpoint-docs

Remove redundant `/v1/logs` path from endpoints in README and example
serilog · Sep 5, 2024 · 87e1ade · 87e1ade
2 parents 2c9ebbc + 3055f99
commit 87e1ade
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/README.md b/README.md
@@ -45,7 +45,7 @@ setups, and looks like:
 ```csharp
 Log.Logger = new LoggerConfiguration()
     .WriteTo.OpenTelemetry(
-        endpoint: "http://127.0.0.1:4318/v1/logs",
+        endpoint: "http://127.0.0.1:4318",
         protocol: OtlpProtocol.HttpProtobuf)
     .CreateLogger();
 ```
@@ -57,7 +57,7 @@ configuration, which looks like:
 Log.Logger = new LoggerConfiguration()
     .WriteTo.OpenTelemetry(options =>
     {
-        options.Endpoint = "http://127.0.0.1:4318/v1/logs";
+        options.Endpoint = "http://127.0.0.1:4318";
         options.Protocol = OtlpProtocol.HttpProtobuf;
     })
     .CreateLogger();

diff --git a/example/Example/Program.cs b/example/Example/Program.cs
@@ -63,7 +63,7 @@ static void SendLogs(ILogger logger, string protocol)
 static Logger CreateLogger(OtlpProtocol protocol)
 {
     var endpoint = protocol == OtlpProtocol.HttpProtobuf ?
-        "http://localhost:4318/v1/logs" :
+        "http://localhost:4318" :
         "http://localhost:4317";
 
     return new LoggerConfiguration()