diff options
| author | Robin Jarry <robin@jarry.cc> | 2023-01-05 12:16:34 +0100 |
|---|---|---|
| committer | Robin Jarry <robin@jarry.cc> | 2023-01-06 23:51:52 +0100 |
| commit | 175c1c822084d0f42892729c8e095397bd830966 (patch) | |
| tree | 332ef46037e29509eb5ee9466ec2e0e2e12e2ee3 /.gitignore | |
| parent | 10b995f0196c8243132c3f435d4e11b3c9700e35 (diff) | |
| download | aerc-175c1c822084d0f42892729c8e095397bd830966.tar.gz | |
contrib: add linter to check for panic handler in goroutines
If log.PanicHandler() is not installed in a goroutine and a panic
occurs, the terminal state is not restored. This causes the panic trace
to be unreadable since the terminal is broken.
Add a custom analyzer that parses our code and ensures that:
defer log.PanicHandler()
is the first statement of all functions that are executed in goroutines.
Include that linter in golangci config.
Signed-off-by: Robin Jarry <robin@jarry.cc>
Tested-by: Bence Ferdinandy <bence@ferdinandy.com>
Diffstat (limited to '.gitignore')
| -rw-r--r-- | .gitignore | 1 |
1 files changed, 1 insertions, 0 deletions
@@ -4,6 +4,7 @@ /aerc.debug /wrap /.aerc.d +/linters.so race.log.* raw.log *.1 |