diff options
| author | Tim Culverhouse <tim@timculverhouse.com> | 2022-09-16 14:41:07 -0500 |
|---|---|---|
| committer | Robin Jarry <robin@jarry.cc> | 2022-09-20 00:03:36 +0200 |
| commit | 01f80721e283086cd122879cb2c9f3bb95b274d7 (patch) | |
| tree | e78f990ac432ed23f2f2941478d7cffa2b416698 /lib/format | |
| parent | 74735711595c3f0dc29177f767b2c91beef19617 (diff) | |
| download | aerc-01f80721e283086cd122879cb2c9f3bb95b274d7.tar.gz | |
msgstore: post MessageInfo on erroneous fetch
When errors occur during a fetch header request, the requested headers
are deleted from pending and no information is given to the UI. Spinners
keep spinning, and ultimately as the view is refreshed, the headers are
fetched again. This can lead to infinite loops, and extremely long logs.
Update the store with a MessageInfo message when an error is received.
Have the UI display that the header couldn't be fetched in the message
list.
Signed-off-by: Tim Culverhouse <tim@timculverhouse.com>
Acked-by: Robin Jarry <robin@jarry.cc>
Diffstat (limited to 'lib/format')
| -rw-r--r-- | lib/format/format.go | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/lib/format/format.go b/lib/format/format.go index 42dd34a1..055e2c73 100644 --- a/lib/format/format.go +++ b/lib/format/format.go @@ -83,6 +83,10 @@ func ParseMessageFormat(format string, timeFmt string, thisDayTimeFmt string, thisWeekTimeFmt string, thisYearTimeFmt string, ctx Ctx) ( string, []interface{}, error, ) { + if ctx.MsgInfo.Error != nil { + return "", nil, + errors.New("(unable to fetch header)") + } retval := make([]byte, 0, len(format)) var args []interface{} |