-
Notifications
You must be signed in to change notification settings - Fork 26
Closed
Labels
Needs: Attention 👋WIPhelp wantedGood candidate for a pull requestGood candidate for a pull requesttype:bugA broken experienceA broken experience
Description
When logging request information, the http.port should only be set if url.port is not None.
| "http.port": url.port, |
Currently, if the
http.port is None, opentelemetry produces the following warning:
Invalid type NoneType for attribute 'http.port' value. Expected one of ['bool', 'str', 'bytes', 'int', 'float'] or a sequence of those types
based on this line of code:
https://github.com/open-telemetry/opentelemetry-python/blob/29aad2efa30a35757ea78d90e4d3fd517bc00468/opentelemetry-api/src/opentelemetry/attributes/__init__.py#L100
See pydantic/logfire#749 for more details.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Needs: Attention 👋WIPhelp wantedGood candidate for a pull requestGood candidate for a pull requesttype:bugA broken experienceA broken experience
Type
Projects
Status
Done ✔️