From 0472f8cfa268b0d735468409cba2221435a2af6b Mon Sep 17 00:00:00 2001
From: Tigran Najaryan <4194920+tigrannajaryan@users.noreply.github.com>
Date: Mon, 16 Nov 2020 14:33:01 -0500
Subject: [PATCH] Change default OTLP port number (#1221)

* Change default OTLP port number

Contributes to https://github.com/open-telemetry/opentelemetry-specification/issues/1148

Note that a separate port is used for OTLP/HTTP for now. There is currently work
in progress to confirm that we can use the same port. Once we have the confirmation
I will update the spec again to use one port.

* Address PR comments
---
 specification/protocol/otlp.md | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/specification/protocol/otlp.md b/specification/protocol/otlp.md
index c2342343d..00e7db36d 100644
--- a/specification/protocol/otlp.md
+++ b/specification/protocol/otlp.md
@@ -289,7 +289,7 @@ some in beta.
 
 #### Default Port
 
-The default network port for OTLP/gRPC is 55680.
+The default network port for OTLP/gRPC is 4317.
 
 ### OTLP/HTTP
 
@@ -445,10 +445,7 @@ connections SHOULD be configurable.
 
 #### Default Port
 
-The default network port for OTLP/HTTP is 55681. There is currently an [open
-issue](https://github.com/open-telemetry/opentelemetry-collector/issues/1256) to
-use the same port for OTLP/gRPC and OTLP/HTTP. In that case this spec will be
-updated to use the same default port for OTLP/gRPC and OTLP/HTTP.
+The default network port for OTLP/HTTP is 4317.
 
 ## Implementation Recommendations