8000 SMTP is uppercase in Windows php.ini directive · Issue #40899 · symfony/symfony · GitHub
[go: up one dir, main page]
Skip to content
8000

SMTP is uppercase in Windows php.ini directive #40899

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.

Sign up for GitHub

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

Jump to bottom
Closed
widhold opened this issue Apr 21, 2021 · 1 comment
Closed

SMTP is uppercase in Windows php.ini directive #40899

widhold opened this issue Apr 21, 2021 · 1 comment
Labels
Bug Mailer Status: Needs Review

Comments

@widhold
Copy link
widhold commented Apr 21, 2021

Symfony version(s) affected: 5.2, symfony/mailer (v5.2.6)

Description
In
vendor\symfony\mailer\Transport\NativeTransportFactory.php line 42

$host = ini_get('smtp'); ... requests the php.ini SMTP environment variable, which is uppercase under windows:
SMTP string
https://www.php.net/manual/en/mail.configuration.php#ini.smtp
... and throws exception!

How to reproduce

Possible Solution
check for uppercase SMTP variable name

Additional context
@widhold widhold added the Bug label Apr 21, 2021
@xabbuh xabbuh mentioned this issue Apr 23, 2021
Merged
@xabbuh
Copy link
Member
xabbuh commented Apr 23, 2021

see #40919

xabbuh added a commit that referenced this issue Apr 23, 2021
@xabbuh
bug #40919 [Mailer] use correct spelling when accessing the SMTP php.…
f1ac7a8 
…ini value (xabbuh)

This PR was merged into the 5.2 branch.

Discussion
----------

[Mailer] use correct spelling when accessing the SMTP php.ini value

| Q             | A
| ------------- | ---
| Branch?       | 5.2
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Tickets       | Fix #40899
| License       | MIT
| Doc PR        |

Commits
-------

b6f8ed6 use correct spelling when accessing the SMTP php.ini value
@xabbuh xabbuh closed this as completed Apr 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Mailer Status: Needs Review
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@derrabus @xabbuh @carsonbot @widhold
0