diff options
| author | Moritz Poldrack <git@moritz.sh> | 2022-03-22 09:52:27 +0100 |
|---|---|---|
| committer | Robin Jarry <robin@jarry.cc> | 2022-03-23 20:56:09 +0100 |
| commit | ae83373fa63883f03bd5580ad3937d1e5fa428ed (patch) | |
| tree | 12e7bae80292088f7dfed45279fa3bce225743bf /commands/account | |
| parent | feecc09b73e2c904be42f271e0c10574b98a95ea (diff) | |
| download | aerc-ae83373fa63883f03bd5580ad3937d1e5fa428ed.zip | |
logging: added a log on panic
Since panics still regularly "destroy" the terminal, it is hard to get a
stack trace for panics you do not anticipate. This commit adds a panic
handler that automatically creates a logfile inside the current working
directory.
It has to be added to every goroutine that is started and will repair
the terminal on a panic.
Signed-off-by: Moritz Poldrack <git@moritz.sh>
Acked-by: Robin Jarry <robin@jarry.cc>
Diffstat (limited to 'commands/account')
| -rw-r--r-- | commands/account/compose.go | 7 | ||||
| -rw-r--r-- | commands/account/recover.go | 7 |
2 files changed, 12 insertions, 2 deletions
diff --git a/commands/account/compose.go b/commands/account/compose.go index 1a62d0a..4884b11 100644 --- a/commands/account/compose.go +++ b/commands/account/compose.go @@ -5,6 +5,7 @@ import ( "regexp" "strings" + "git.sr.ht/~rjarry/aerc/logging" "git.sr.ht/~rjarry/aerc/models" "git.sr.ht/~rjarry/aerc/widgets" "git.sr.ht/~sircmpwn/getopt" @@ -52,7 +53,11 @@ func (Compose) Execute(aerc *widgets.Aerc, args []string) error { } tab.Content.Invalidate() }) - go composer.AppendContents(strings.NewReader(body)) + go func() { + defer logging.PanicHandler() + + composer.AppendContents(strings.NewReader(body)) + }() return nil } diff --git a/commands/account/recover.go b/commands/account/recover.go index 8158af6..165e88e 100644 --- a/commands/account/recover.go +++ b/commands/account/recover.go @@ -8,6 +8,7 @@ import ( "path/filepath" "git.sr.ht/~rjarry/aerc/commands" + "git.sr.ht/~rjarry/aerc/logging" "git.sr.ht/~rjarry/aerc/models" "git.sr.ht/~rjarry/aerc/widgets" "git.sr.ht/~sircmpwn/getopt" @@ -111,7 +112,11 @@ func (Recover) Execute(aerc *widgets.Aerc, args []string) error { tab.Name = subject tab.Content.Invalidate() }) - go composer.AppendContents(bytes.NewReader(data)) + go func() { + defer logging.PanicHandler() + + composer.AppendContents(bytes.NewReader(data)) + }() // remove file if force flag is set if force { |