Skip to content
This repository has been archived by the owner on Feb 6, 2022. It is now read-only.

Typo fix #281

Merged
merged 1 commit into from Apr 21, 2019
Merged

Typo fix #281

merged 1 commit into from Apr 21, 2019

Conversation

cybernet
Copy link
Contributor

No description provided.

@fabpot
Copy link
Member

fabpot commented Apr 21, 2019

Good catch, thanks @cybernet.

@fabpot fabpot merged commit 2bb072e into symfony:master Apr 21, 2019
fabpot added a commit that referenced this pull request Apr 21, 2019
This PR was merged into the 3.2-dev branch.

Discussion
----------

Typo fix

Commits
-------

2bb072e Typo fix
@cybernet
Copy link
Contributor Author

cybernet commented Apr 21, 2019

I've found this error because i'm trying to configure the bundle as per docs with aws ses
like - smtp://usr:pass/wd@email-smtp.eu-west-1.amazonaws.com:465?encryption=ssl

note: my generated password has "/"

this might be triggering the error

and i get this

at Symfony\Bundle\SwiftmailerBundle\DependencyInjection\SwiftmailerTransportFactory::resolveOptions(array('transport' => 'smtp', 'url' => 'smtp://usr:pass/wd@email-smtp.eu-west-1.amazonaws.com:465?encryption=ssl', 'username' => null, 'password' => null, 'host' => 'localhost', 'port' => null, 'timeout' => 30, 'source_ip' => null, 'local_domain' => null, 'encryption' => null, 'auth_mode' => null, 'command' => '/usr/sbin/sendmail -bs'))

so now i'm thinking if i got it set wrong

obviously i removed and tried again - same error ... so what am I not getting ?

I'm happy to make it an issue report

also tried to 'escape' it ... same problem ...

any idea ? 😁

@naderman
Copy link

@cybernet You need to urlencode username and password. It still didn't work because of a bug that was fixed here: #263 - for now the workaround is to use query params, e.g. ?username=usr&password=pass%2Fwd

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants