Skip to content

Commit

Permalink
Clarify that client.port is the port of whichever client was captur…
Browse files Browse the repository at this point in the history
…ed in `client.address` (#471)
  • Loading branch information
trask authored Oct 30, 2023
1 parent a1ec477 commit de4125d
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 5 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,8 @@ release.
- Change `http.request.body.size` and `http.response.body.size`
from recommended to opt-in.
([#460](https://github.com/open-telemetry/semantic-conventions/pull/460))
- Clarify that `client.port` is the port of whichever client was captured in `client.address`.
([#471](https://github.com/open-telemetry/semantic-conventions/pull/471))

### Features

Expand Down
2 changes: 1 addition & 1 deletion docs/http/http-spans.md
Original file line number Diff line number Diff line change
Expand Up @@ -341,7 +341,7 @@ For an HTTP server span, `SpanKind` MUST be `Server`.
| Attribute | Type | Description | Examples | Requirement Level |
|---|---|---|---|---|
| [`client.address`](../general/attributes.md) | string | Client address - domain name if available without reverse DNS lookup; otherwise, IP address or Unix domain socket name. [1] | `83.164.160.102` | Recommended |
| [`client.port`](../general/attributes.md) | int | The port of the original client behind all proxies, if known (e.g. from [Forwarded#for](https://developer.mozilla.org/docs/Web/HTTP/Headers/Forwarded#for) or a similar header). Otherwise, the immediate client peer port. [2] | `65123` | Recommended |
| [`client.port`](../general/attributes.md) | int | The port of whichever client was captured in `client.address`. [2] | `65123` | Recommended |
| [`http.route`](../attributes-registry/http.md) | string | The matched route, that is, the path template in the format used by the respective server framework. [3] | `/users/:userID?`; `{controller}/{action}/{id?}` | Conditionally Required: If and only if it's available |
| [`network.local.address`](../attributes-registry/network.md) | string | Local socket address. Useful in case of a multi-IP host. | `10.1.2.80`; `/tmp/my.sock` | Opt-In |
| [`network.local.port`](../attributes-registry/network.md) | int | Local socket port. Useful in case of a multi-port host. | `65123` | Opt-In |
Expand Down
5 changes: 1 addition & 4 deletions model/trace/http.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -71,10 +71,7 @@ groups:
Otherwise, the immediate client peer address.
examples: ['83.164.160.102']
- ref: client.port
brief: >
The port of the original client behind all proxies, if
known (e.g. from [Forwarded#for](https://developer.mozilla.org/docs/Web/HTTP/Headers/Forwarded#for) or a similar header).
Otherwise, the immediate client peer port.
brief: The port of whichever client was captured in `client.address`.
- ref: url.path
requirement_level: required
sampling_relevant: true
Expand Down

0 comments on commit de4125d

Please sign in to comment.