teslamotors / fleet-telemetry

Apache License 2.0

625 stars 66 forks source link

Fix airbrake notice message #149

Closed agbpatro closed 2 months ago

agbpatro commented 2 months ago

Description

Errbit logs error like this

Screenshot 2024-04-08 at 13 31 49

This format isn't useful. Making following changes

logType is int. logrus.AllLogType[logType] will return value in string
err.Error() extracts error message to be logged properly in airbrake

Type of change

Please select all options that apply to this change:

[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[X] Bug fix (non-breaking change which fixes an issue)
[ ] Documentation update

Checklist:

Confirm you have completed the following steps:

[X] My code follows the style of this project.
[X] I have performed a self-review of my code.
[ ] I have made corresponding updates to the documentation.
[X] I have added/updated unit tests to cover my changes.
[ ] I have added/updated integration tests to cover my changes.