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 way to set sslValidation for smtp connection in sendMail #528

Merged
merged 1 commit into from Feb 16, 2014

Conversation

Projects
None yet
2 participants
@Extrawurst
Contributor

Extrawurst commented Feb 14, 2014

No description provided.

@s-ludwig

This comment has been minimized.

Show comment
Hide comment
@s-ludwig

s-ludwig Feb 16, 2014

Member

Thanks, a setup callback similar to the HTTPClient.setSSLSetupCallback will also be necessary to use this in practice (the trusted certificates must be added to the SSLContext). Do you want to add that, too, or leave it for later?

Member

s-ludwig commented Feb 16, 2014

Thanks, a setup callback similar to the HTTPClient.setSSLSetupCallback will also be necessary to use this in practice (the trusted certificates must be added to the SSLContext). Do you want to add that, too, or leave it for later?

@Extrawurst

This comment has been minimized.

Show comment
Hide comment
@Extrawurst

Extrawurst Feb 16, 2014

Contributor

Can't promise to do that very soon since I am very cluessless about the whole ssl stuff. For our use case this option was enough, since we just wanted to have the old behaviour working again.

Contributor

Extrawurst commented Feb 16, 2014

Can't promise to do that very soon since I am very cluessless about the whole ssl stuff. For our use case this option was enough, since we just wanted to have the old behaviour working again.

@s-ludwig

This comment has been minimized.

Show comment
Hide comment
@s-ludwig

s-ludwig Feb 16, 2014

Member

Okay, that's fine, just wanted to verify. I'll do that on occasion, it's just three lines or something.

Member

s-ludwig commented Feb 16, 2014

Okay, that's fine, just wanted to verify. I'll do that on occasion, it's just three lines or something.

s-ludwig added a commit that referenced this pull request Feb 16, 2014

Merge pull request #528 from Extrawurst/master
add way to set sslValidation for smtp connection in sendMail

@s-ludwig s-ludwig merged commit 3175706 into vibe-d:master Feb 16, 2014

1 check passed

default The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment