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

feat(transport): add sendmail transport #83

Merged
merged 2 commits into from
Oct 23, 2016
Merged

feat(transport): add sendmail transport #83

merged 2 commits into from
Oct 23, 2016

Conversation

paradoxix
Copy link

Addresses: Sendmail transport support #40

@GitCop
Copy link

GitCop commented Oct 14, 2016

There were the following issues with your Pull Request

  • Commit: 04f54f3
    • Invalid type. Valid types are feat, fix, docs, style, refactor, perf, test, chore

Guidelines are available at https://github.com/lettre/lettre/blob/master/CONTRIBUTING.md


This message was auto-generated by https://gitcop.com

@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.196% when pulling 1cb499f on paradoxix:sendmail into d944aed on lettre:master.

@amousset amousset changed the title feat: add sendmail transport feat(transport): add sendmail transport Oct 19, 2016
@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.196% when pulling 21fc70c on paradoxix:sendmail into 5be0f86 on lettre:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.196% when pulling 13ee61d on paradoxix:sendmail into 5be0f86 on lettre:master.

Copy link
Member

@amousset amousset left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We also need a section in the usage doc in lib.rs for this new transport.
This is a nice improvement, thanks!

pub mod error;

/// Writes the content and the envelope information to a file
pub struct SendmailTransport;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we need to make the command to use configurable, and store it in SendmailTransport.


impl EmailTransport<SendmailResult> for SendmailTransport {
fn send<T: SendableEmail>(&mut self, email: T) -> SendmailResult {
// Spawn the `wc` command
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This comment should be removed.

@amousset amousset merged commit 13ee61d into lettre:master Oct 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

4 participants