From 7711980c94daaa62fd6471c47f34a04cfcdb2c88 Mon Sep 17 00:00:00 2001
From: Tigran Najaryan <4194920+tigrannajaryan@users.noreply.github.com>
Date: Tue, 3 Aug 2021 12:05:31 +0400
Subject: [PATCH] Change OTLP/HTTP port from 4317 to 4318 (#1839)

Related to https://github.com/open-telemetry/opentelemetry-specification/issues/1816
Fixes https://github.com/open-telemetry/opentelemetry-specification/issues/1835

Some historical context: we wanted to make grpc and http use the same port and we
had an open issue in the Collector to do so:
https://github.com/open-telemetry/opentelemetry-collector/issues/1256

The conclusion is that there are technical hurdles that make it unfeasible:
https://github.com/open-telemetry/opentelemetry-collector/issues/1256#issuecomment-880048833

Because of that we need to keep grpc and http ports separate. This means we need to
change the spec to say that otlp/http uses port 4318. Once this PR is merged we
will also need to submit for port 4318 registration with IANA like we did previously
with port 4317:
https://github.com/open-telemetry/opentelemetry-specification/issues/1148#issuecomment-725067404
---
 specification/protocol/otlp.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/specification/protocol/otlp.md b/specification/protocol/otlp.md
index 6c83e2dc5..16ffe30fd 100644
--- a/specification/protocol/otlp.md
+++ b/specification/protocol/otlp.md
@@ -473,7 +473,7 @@ connections SHOULD be configurable.
 
 #### OTLP/HTTP Default Port
 
-The default network port for OTLP/HTTP is 4317.
+The default network port for OTLP/HTTP is 4318.
 
 ## Implementation Recommendations