Skip to content

Commit ba90a1f

Browse files
author
Liudmila Molkova
committed
minor fixes
1 parent 569313a commit ba90a1f

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

oteps/4333-recording-exceptions-on-logs.md

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,12 @@ This guidance boils down to the following:
3535

3636
> [!NOTE]
3737
>
38-
> Based on this guidance non-native instrumentations should record exceptions in top-level instrumentations only (#2 above)
38+
> Based on this guidance non-native instrumentations should record exceptions in top-level instrumentations only (#2 in [Details](#details))
39+
40+
> [!Important]
41+
>
42+
> OTel should provide APIs like `setException` when creating log record that will record only necessary information depending
43+
> on the configuration and log severity. See [API changes](#api-changes) for the details.
3944
4045
### Details
4146

@@ -79,7 +84,7 @@ Library may write logs providing exception instance through a log bridge and not
7984
It also maybe desirable by some vendors/apps to record all the exception details.
8085

8186
OTel Logs API should provide additional methods that enrich log record with exception details such as
82-
`addException(exception)` (`addUnhandledException`, etc), similar to [RecordException](../specification/trace/api.md?plain=1#L682)
87+
`setException(exception)` (`setUnhandledException`, etc), similar to [RecordException](../specification/trace/api.md?plain=1#L682)
8388
method on span.
8489

8590
OTel SDK should implement such methods and set exception attributes based on configuration

0 commit comments

Comments
 (0)