Skip to content
This repository was archived by the owner on Oct 3, 2023. It is now read-only.

Commit 93a4f33

Browse files
rakyllBogdan Drutu
authored andcommitted
Remove port from HTTP trace attributes (#76)
Host already contains the port, we don't need to record it separately.
1 parent 578e5ad commit 93a4f33

1 file changed

Lines changed: 1 addition & 3 deletions

File tree

trace/HTTP.md

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -83,8 +83,7 @@ All attributes are optional.
8383

8484
| Attribute name | Description | Example value |
8585
|---------------------------|-----------------------------|---------------------------------|
86-
| "http.host" | Request URL host | "example.com" |
87-
| "http.port" | Request URL port | 443 |
86+
| "http.host" | Request URL host | "example.com:779" |
8887
| "http.method" | Request URL method | "GET" |
8988
| "http.path" | Request URL path | "/users/25f4c31d" |
9089
| "http.route" | Matched request URL route | "/users/:userID" |
@@ -103,7 +102,6 @@ known attributes/labels on supported tracing backends.
103102
| "http.host" | "http.host" | "http.host" | "/http/host" |
104103
| "http.method" | "http.method" | "http.method" | "/http/method" |
105104
| "http.path" | "http.path" | "http.path" | "/http/path" |
106-
| "http.port" | "http.port" | "http.port" | "http.port" |
107105
| "http.route" | "http.route" | "http.route" | "/http/route" |
108106
| "http.user_agent" | "http.user_agent" | "http.user_agent" | "/http/user_agent" |
109107
| "http.status_code" | "http.status_code" | "http.status_code" | "/http/status_code" |

0 commit comments

Comments
 (0)