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

SMTP over TLS does not work #122

Closed
kazu-yamamoto opened this issue Apr 15, 2019 · 3 comments
Closed

SMTP over TLS does not work #122

kazu-yamamoto opened this issue Apr 15, 2019 · 3 comments

Comments

@kazu-yamamoto
Copy link
Owner

This is due to #121.
I'm now trying to fix this.

kazu-yamamoto added a commit that referenced this issue Apr 15, 2019
@kazu-yamamoto
Copy link
Owner Author

@hrs-allbsd Would you check if 70d6da0 maintain your intentional behavior?

@hrs-allbsd
Copy link
Contributor

@hrs-allbsd Would you check if 70d6da0 maintain your intentional behavior?

Yes, thanks for fixing it. My patch did not honor the assumption that (mew-serv-to-port) always resolved the service name to a number before entering into (mew-open-network-stream). The (stringp port) check was needed before (string-match) as in your change.

@kazu-yamamoto
Copy link
Owner Author

Thanks. Let's close this.

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

No branches or pull requests

2 participants