#251 panic when fetching message headers a month ago

Comment by ~pmolloy on ~sircmpwn/aerc2

This is still an issue on 12/27/2019 when building off of master -> 6794ce0

#240 Crash when console becomes too small 5 months ago

Comment by ~pmolloy on ~sircmpwn/aerc2

Looks like this is a duplicate of #99

#270 Reply-all (rr) crashes on OpenBSD 5 months ago

Comment by ~pmolloy on ~sircmpwn/aerc2

Neither do I! It looks like errors returned by gomail.ParseAddress() on line 61 are not handled. In that case if the first value is nil it would explain the panic on line 97 when it is dereferenced.

If you're building from source you could change line 61 to something like the following:

fmt.Println(conf.From)
us, err := gomail.ParseAddress(conf.From)
if us == nil {
    fmt.Println(err)
    return err
}

#271 panic in pty.StartWithSize 5 months ago

Comment by ~pmolloy on ~sircmpwn/aerc2

Looks like this was fixed by 5493af8c on 8/12. You could test it by checking out master and building from source.

#277 Panic while navigating folders 5 months ago

Comment by ~pmolloy on ~sircmpwn/aerc2

It looks like there is an assumption that MessageStore.Update() is first called with an argument of type DirectoryContents (see worker/types/message.go). Otherwise the Messages attribute of the MessageStore struct won't get initialized since it is only initialized in that case.

#251 panic when fetching message headers 6 months ago

Comment by ~pmolloy on ~sircmpwn/aerc2

It looks like this was introduced by b83e7c9fa6a0d187a0f20d98d522cff792053cdd on Fri Jun 7 13:56:14 2019. Note the error returned by textproto.ReaderHeader() is EOF.

#251 panic when fetching message headers 6 months ago

Comment by ~pmolloy on ~sircmpwn/aerc2

msg.RFC822Header is nil when passed to forMessage(). I was able to grep around and in worker/imap/fetch.go within handleFetchMessage() perhaps unsurprisingly header is also nil. It looks like textproto.Reader() is returning an error and header is only assigned something if there is no error. Basically the error is not handled.

#251 panic when fetching message headers 6 months ago

Comment by ~pmolloy on ~sircmpwn/aerc2

I just ran into the same issue using protonmail-bridge v1.1.6