diff options
| author | Robin Jarry <robin@jarry.cc> | 2024-07-02 15:58:41 +0200 |
|---|---|---|
| committer | Robin Jarry <robin@jarry.cc> | 2024-07-02 22:12:45 +0200 |
| commit | 44b14ebdff0e3da8df2348eeda27bfc2ec421fbb (patch) | |
| tree | 8711abffa76e415dde14a67a671603fa27da586f /lib/rfc822/message.go | |
| parent | bee2fc62ac7808cf081e58947023cd4c61c0109e (diff) | |
| download | aerc-44b14ebdff0e3da8df2348eeda27bfc2ec421fbb.tar.gz | |
rfc822: be liberal with invalid address headers
Some email clients format email addresses with quotes around B encoded
names. E.g.:
"=?utf-8?B?U21pZXRhbnNraSwgV29qY2llY2ggVGFkZXVzeiBpbiBUZWFtcw==?="
This seems non standard. I tried reading the multiple RFCs that describe
the MIME headers encoding but could not find any precise answer to
determine if adding quotes around encoded words is valid or not.
In any case, ParseAddressList completely ignores any encoding when it
encounters a double quote.
When that happens, try to detect it and force decoding a second time
after the quotes have been removed.
Link: https://datatracker.ietf.org/doc/html/rfc2045#section-3
Link: https://datatracker.ietf.org/doc/html/rfc2047#section-2
Link: https://datatracker.ietf.org/doc/html/rfc822#section-3.3
Reported-by: Bence Ferdinandy <bence@ferdinandy.com>
Signed-off-by: Robin Jarry <robin@jarry.cc>
Tested-by: Bence Ferdinandy <bence@ferdinandy.com>
Diffstat (limited to 'lib/rfc822/message.go')
| -rw-r--r-- | lib/rfc822/message.go | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/lib/rfc822/message.go b/lib/rfc822/message.go index 391ab3e5..26d790ff 100644 --- a/lib/rfc822/message.go +++ b/lib/rfc822/message.go @@ -6,6 +6,7 @@ import ( "errors" "fmt" "io" + "mime" "regexp" "strings" "time" @@ -284,6 +285,13 @@ func parseAddressList(h *mail.Header, key string) ([]*mail.Address, error) { // errors which are not fatal. return nil, err } + for _, addr := range addrs { + // Handle invalid headers with quoted *AND* encoded names + if strings.HasPrefix(addr.Name, "=?") && strings.HasSuffix(addr.Name, "?=") { + d := mime.WordDecoder{CharsetReader: message.CharsetReader} + addr.Name, _ = d.DecodeHeader(addr.Name) + } + } // If we got at least one address, ignore any returned error. return addrs, nil } |