Try to fix weird sender names: High level patch idea

[benhoff]

Hey, I haven’t tested this locally since I don’t have a dev environment set up, but I wanted to put this up because I was hitting a problem importing PSTs.

When I try to import from a PST, the sender names look like this:

From: /O=EXCHANGELABS/OU=EXCHANGE ADMINISTRATIVE GROUP (************)/CN=RECIPIENTS/CN=********-********

That string is the Exchange LegacyDN (an internal X.500 identifier), not an SMTP address. It happens because when senderAddrtype === "EX", the MAPI properties PR_SENDER_EMAIL_ADDRESS or PR_EMAIL_ADDRESS only return the LegacyDN, not the real SMTP.

What this patch changes

From header:

• Prefer SMTP if addrType === "SMTP" and the address looks like an email.
• Otherwise, try to extract a From: line from PR_TRANSPORT_MESSAGE_HEADERS.
• As a last resort, fall back to just the display name (but no ).
• Sanitize names and RFC-2047 encode if non-ASCII.

Recipients (To/Cc/Bcc):

• Build these from the recipient table rather than the displayTo/displayCc strings.
• Use smtpAddress (or emailAddress when addrType === "SMTP").
• Skip recipients without SMTP addresses so we don’t end up with names-only.

Why

This should avoid weird LegacyDNs showing up in headers and instead generate proper RFC-2822 headers like:

From: Ben <ben@company.com>
To: Jane Doe <jane.doe@company.com>

instead of leaking raw X.500 strings.

It needs to be thoroughly tested though.

If this is something that you would be open to testing, I would be grateful. As I'm not familiar with the buildstack, it would be difficult for me to get up and running.

[github-actions]

All contributors have signed the CLA ✍️ ✅
_{Posted by the CLA Assistant Lite bot.}

[benhoff]

I have read the CLA Document and I hereby sign the CLA