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

Port ignored in Mailer._send #43

Closed
00ff opened this Issue Aug 4, 2013 · 1 comment

Comments

Projects
None yet
2 participants
@00ff

00ff commented Aug 4, 2013

The port is properly parsed from the email_url, but then it's not used. AFAICT, there's really no way starttls would work without it. On my local site-packages version, I changed it to be:

        if proto == 'ssl':
            log.debug("Setting up SSL")
            session = SMTP_SSL(self._conf['fqdn'], self._conf['port'])
        else:
            session = SMTP(self._conf['fqdn'], self._conf['port'])

Sorry I'm too busy to submit a proper patch.

@00ff

This comment has been minimized.

Show comment
Hide comment
@00ff

00ff Aug 4, 2013

And just an fyi, I tested this and it worked with gmail starttls smtp server.

00ff commented Aug 4, 2013

And just an fyi, I tested this and it worked with gmail starttls smtp server.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment