diff options
| author | Aleksandr Nogikh <nogikh@google.com> | 2022-11-17 16:11:21 +0000 |
|---|---|---|
| committer | Aleksandr Nogikh <wp32pw@gmail.com> | 2022-11-21 16:36:47 +0100 |
| commit | b0668b991fa57cd3934dc5e93ddef45e56a3462c (patch) | |
| tree | 667333b75f2e081a17b88c5a40a371e161c01453 /pkg/email/parser.go | |
| parent | ed183554eb19dc7e173e1952381115c816883437 (diff) | |
dashboard: reduce CanonicalEmail calls
Sometimes we call it before accessing emails, sometimes we do not. This
is prone to errors.
Always canonicalize From and Cc, this should somewhat reduce the problem.
Diffstat (limited to 'pkg/email/parser.go')
| -rw-r--r-- | pkg/email/parser.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/pkg/email/parser.go b/pkg/email/parser.go index 2975d2de7..6c0fb22d4 100644 --- a/pkg/email/parser.go +++ b/pkg/email/parser.go @@ -94,7 +94,7 @@ func Parse(r io.Reader, ownEmails []string) (*Email, error) { bugID = context } } else { - ccList = append(ccList, cleaned) + ccList = append(ccList, CanonicalEmail(cleaned)) } } ccList = MergeEmailLists(ccList) @@ -138,7 +138,7 @@ func Parse(r io.Reader, ownEmails []string) (*Email, error) { MessageID: msg.Header.Get("Message-ID"), Link: link, Subject: subject, - From: from[0].String(), + From: CanonicalEmail(from[0].Address), Cc: ccList, Sender: sender, Body: bodyStr, |