Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for setting dsn per email in smpt client #136

Closed
jagtyp opened this issue Dec 15, 2014 · 5 comments
Closed

Add support for setting dsn per email in smpt client #136

jagtyp opened this issue Dec 15, 2014 · 5 comments
Labels
enhancement New feature or request

Comments

@jagtyp
Copy link

jagtyp commented Dec 15, 2014

I would want to be able to set per email which delivery notifications should be sent back to the return-path in the smtp client.

They describe in below rfc's how this should be implemented:
https://www.ietf.org/rfc/rfc3461.txt
https://www.ietf.org/rfc/rfc1891.txt
It's possible to do quite alot here, but for me simply setting per email (per send) would be just fine.

Thanks
/David

jstedfast added a commit that referenced this issue Dec 16, 2014
@jstedfast jstedfast added the enhancement New feature or request label Dec 16, 2014
@jstedfast
Copy link
Owner

I've implemented support for this now, but currently you'd need to subclass SmtpClient to use it (and override GetDeliveryStatusNotifications). I also implemented support for the ENVID param since that looked useful for anyone writing an automated system to track DSN's.

If that's good enough for your needs, I can close this and call it done. Otherwise I can possibly look into adding more Send() overloads, I would just prefer not to if I can help it.

@jagtyp
Copy link
Author

jagtyp commented Dec 17, 2014

Thanks, this looks great. I'll take a look at it tomorrow. It's seems to be what I was looking for.

Great work :)

@jagtyp
Copy link
Author

jagtyp commented Dec 19, 2014

Hi. I tested it yesterday, and it's working just fine.
You could close this issue.

Thanks again :)

@jstedfast
Copy link
Owner

I just released MailKit 1.0.5 to NuGet this morning with this fix.

@jagtyp
Copy link
Author

jagtyp commented Jan 9, 2015

Hi.

Thanks, then I'll update the packages to use nuget instead.

Best regards

David Clarke
Developer, Bosbec AB

Cell: +46 (0) 735 - 09 00 65
E-mail: david@bosbec.se

Bosbec AB
Klammerdammsgatan 29, 302 42 Halmstad
Phone: +46 (0) 35 - 22 79 60

Head office:
Färögatan 33, Plan 31 Kista Science Tower
SE - 164 51, Kista, Sweden
Phone: +46 (0) 8 - 22 25 25

http://www.bosbec.se/

Jeffrey Stedfast skrev den 2015-01-08 16:15:

I just released MailKit 1.0.5 to NuGet this morning with this fix.


Reply to this email directly or view it on GitHub
#136 (comment).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants