Fix server.port requirement level

CHANGELOG.md

@@ -21,6 +21,8 @@ release.
   ([#410](https://github.com/open-telemetry/semantic-conventions/pull/410))
 - BREAKING: Factor in `X-Forwarded-Host` / `Forwarded` when capturing `server.address` and `server.port`.
   ([#411](https://github.com/open-telemetry/semantic-conventions/pull/411))
+- Fix `server.port` to be not required when `server.address` is not set.
+  ([#429](https://github.com/open-telemetry/semantic-conventions/pull/429))
 
 ### Features
 

docs/http/http-spans.md

@@ -391,7 +391,7 @@ MUST NOT include the port identifier.
   [X-Forwarded-Host](https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/X-Forwarded-Host), or a similar header.
 - Port identifier of the `Host` header
 
-**[6]:** If not default (`80` for `http` scheme, `443` for `https`).
+**[6]:** If `server.address` is set and the port is not default (`80` for `http` scheme, `443` for `https`).
 
 **[7]:** When missing, the value is assumed to be `/`
 

model/http-common.yaml

@@ -98,7 +98,7 @@ groups:
             [X-Forwarded-Host](https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/X-Forwarded-Host), or a similar header.
           - Port identifier of the `Host` header
         requirement_level:
-          conditionally_required: If not default (`80` for `http` scheme, `443` for `https`).
+          conditionally_required: If `server.address` is set and the port is not default (`80` for `http` scheme, `443` for `https`).
       - ref: url.scheme
         requirement_level: required
         examples: ["http", "https"]