Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

[Mailer] Set default crypto method#31846

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
fabpot merged 1 commit intosymfony:4.3frombpolaszek:mailer-crypto-type
Jun 5, 2019
Merged

[Mailer] Set default crypto method#31846

fabpot merged 1 commit intosymfony:4.3frombpolaszek:mailer-crypto-type
Jun 5, 2019

Conversation

@bpolaszek
Copy link
Contributor

@bpolaszekbpolaszek commentedJun 4, 2019•
edited
Loading

QA
Branch?4.3
Bug fix?yes
New feature?no
BC breaks?no
Deprecations?no
Tests pass?yes
Fixed tickets#31105
LicenseMIT
Doc PR-

This PRfixes#31105 by providingSTREAM_CRYPTO_METHOD_TLS_CLIENT | STREAM_CRYPTO_METHOD_TLSv1_2_CLIENT | STREAM_CRYPTO_METHOD_TLSv1_1_CLIENT as default crypto method when none is defined in user options and TLS is enabled.

@bpolaszekbpolaszek changed the titleSet default crypto method - Fix #31105[Mailer] Set default crypto method - Fix #31105Jun 4, 2019
@bpolaszekbpolaszek changed the title[Mailer] Set default crypto method - Fix #31105[Mailer] Set default crypto methodJun 4, 2019
@fabpot
Copy link
Member

@bpolaszek Looks like an unrelated commit is part of your PR now (from a PR I've just merged).

@bpolaszek
Copy link
ContributorAuthor

@fabpot Yes, since travis and appveyor failed on my 2nd commit, I tried to rebase and it looks like it's all messed up... I'm trying again.

@bpolaszek
Copy link
ContributorAuthor

Well, builds are still failing but I don't think it's related.

@fabpot
Copy link
Member

Thank you@bpolaszek.

bpolaszek reacted with thumbs up emoji

@fabpotfabpot merged commit4f0ad25 intosymfony:4.3Jun 5, 2019
fabpot added a commit that referenced this pull requestJun 5, 2019
This PR was merged into the 4.3 branch.Discussion----------[Mailer] Set default crypto method| Q             | A| ------------- | ---| Branch?       | 4.3| Bug fix?      | yes| New feature?  | no| BC breaks?    | no| Deprecations? | no| Tests pass?   | yes| Fixed tickets |#31105| License | MIT| Doc PR | -This PRfixes#31105 by providing `STREAM_CRYPTO_METHOD_TLS_CLIENT | STREAM_CRYPTO_METHOD_TLSv1_2_CLIENT | STREAM_CRYPTO_METHOD_TLSv1_1_CLIENT` as  default crypto method when none is defined in user options and TLS is enabled.Commits-------4f0ad25 Set default crypto method -Fix#31105
@fabpotfabpot mentioned this pull requestJun 6, 2019
@xavismeh
Copy link
Contributor

Works now like a charm using Amazon SES transport — which relays on (e)smtp transport. Thank you@bpolaszek !

bpolaszek reacted with thumbs up emoji

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

Reviewers

@stofstofstof left review comments

@fabpotfabpotfabpot approved these changes

Assignees

No one assigned

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@bpolaszek@fabpot@xavismeh@stof@carsonbot
[8]ページ先頭
©2009-2025 Movatter.jp