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

use port 587 in Amazon SES example #6510

Merged
merged 1 commit into from
Apr 24, 2016
Merged

use port 587 in Amazon SES example #6510

merged 1 commit into from
Apr 24, 2016

Conversation

snoek09
Copy link

@snoek09 snoek09 commented Apr 22, 2016

Q A
Doc fix? yes
New docs? no
Applies to all
Fixed tickets -

Today I tried to get Amazon SES working using the cookbook recipe from http://symfony.com/doc/current/cookbook/email/cloud.html

For some reason port 465 was giving a timeout. Changing it to 587 fixed it.
Unless there is a good reason to keep this value I would like to change this port value.

@xabbuh
Copy link
Member

xabbuh commented Apr 23, 2016

According to their documentation 587 should be used for STARTTLS (while 465 is meant to be used with the TLS wrapper which encrypts the entire SMTP protocol). However, the tls option value tells Swiftmailer to use STARTTLS (see swiftmailer/swiftmailer@12a1510).

Thus 👍 from me for this change.

@xabbuh
Copy link
Member

xabbuh commented Apr 24, 2016

Thank you @snoek09.

@xabbuh xabbuh merged commit 29d7331 into symfony:2.3 Apr 24, 2016
xabbuh added a commit that referenced this pull request Apr 24, 2016
This PR was merged into the 2.3 branch.

Discussion
----------

use port 587 in Amazon SES example

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | all
| Fixed tickets | -

Today I tried to get Amazon SES working using the cookbook recipe from http://symfony.com/doc/current/cookbook/email/cloud.html

For some reason port `465` was giving a timeout. Changing it to `587` fixed it.
Unless there is a good reason to keep this value I would like to change this port value.

Commits
-------

29d7331 use port 587 in Amazon SES example
@snoek09 snoek09 deleted the change-port-in-cloud-email-cookbook branch April 25, 2016 06:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants