Excel-DNA / Excel-DNA/ExcelDna

Include more detailed info into the Logger.Error(Exception ex, string message, params object[] args) method

Open
#411 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
C#
Stars
1.5k
Forks
292
Avg merge
20d 12h
Merged PRs (30d)
1

Description

At the current moment it only includes Exception.Message into the logged message, what isn't much useful for generic exceptions like NullRefException etc.
It would be better to include stack trace, for instance:
```
Log(TraceEventType.Error, "{0} : {1} - {2}", message, ex.GetType().Name, ex.ToString().Replace("\r\n", " \\r\\n "));
```

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.