james/nixery
1
0
Fork 0
mirror of https://code.tvl.fyi/depot.git:/tools/nixery.git synced 2025-03-15 06:01:51 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

refactor(server): Handle non-error errors safely

Browse Source 
This case should not be possible unless something manually constructs
a logrus entry with a non-error value in the log.ErrorKey field, but
it's better to be safe than sorry.
This commit is contained in:
Vincent Ambo 2019-10-28 22:38:27 +01:00 committed by Vincent Ambo
parent 583715adc6
commit 8f48226b24
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
server/logs.go
View File

@ -76,11 +76,12 @@ func (f stackdriverFormatter) Format(e *log.Entry) ([]byte, error) {
msg["eventTime"] = &e.Time msg["eventTime"] = &e.Time
msg["severity"] = logSeverity(e.Level) msg["severity"] = logSeverity(e.Level)
if err, ok := msg[log.ErrorKey]; ok { if e, ok := msg[log.ErrorKey]; ok {
// TODO(tazjin): Cast safely - for now there should be if err, isError := e.(error); isError {
// no calls to `.WithError` with a nil error, but who msg[log.ErrorKey] = err.Error()
// knows. } else {
msg[log.ErrorKey] = (err.(error)).Error() delete(msg, log.ErrorKey)
}
} }
if isError(e) { if isError(e) {