fix: smtp starttls compatibility issue #292
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Issue about not supporting StartTLS with Ahasend |
|
本来之前商业版已经修过这个问题，开源版没有下放。计划将重新投入开源建设 (#285)，这段commit还没来的及cherry pick过来，感谢你提前fix 👍 ！ |
zmh-program
changed the title
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
原先mail.v2实现的StartTLS存在问题，导致其与一部分SMTP服务商无法成功握手，此提交将smtp.go的实现转换为gomail.v2以解决StartTLS相关问题，我已分别测试过一家使用StartTLS的服务商(ahasend)和一家使用SSL的服务商(阿里云)。