Skip to content

Conversation

@RoyalBingBong
Copy link
Contributor

@RoyalBingBong RoyalBingBong commented Feb 28, 2024

In our application we need to handle MDNs and for that we need to parse the "Disposition-Notification-To" header and compare the address with the Return-Path header.

I would like to have the same structure of addresses and use mail parser function, but it feels quite dirty to use undocumented and seemingly internal-only methods like decodeAddresses, getAddressesHTML and getAddressesText, so I decided to add a small PR in the hopes that it will be accepted and merged in a timely manner.

Unfortunately I do not have the time (and quite frankly the knowledge) to implement all the parsing related to MDNs (e.g. machine readable and human readable content for an MDN mail).

@CLAassistant
Copy link

CLAassistant commented Feb 28, 2024

CLA assistant check
All committers have signed the CLA.

@andris9 andris9 merged commit 87c32a1 into nodemailer:master Feb 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants