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

The "mailjet+smtp" scheme is not supported. #38316

Closed
sandoche opened this issue Sep 27, 2020 · 6 comments · Fixed by #38348
Closed

The "mailjet+smtp" scheme is not supported. #38316

sandoche opened this issue Sep 27, 2020 · 6 comments · Fixed by #38348
Labels
Bug Good first issue Ideal for your first contribution! (some Symfony experience may be required) Help wanted Issues and PRs which are looking for volunteers to complete them. Mailer Status: Needs Review

Comments

@sandoche
Copy link

sandoche commented Sep 27, 2020

Symfony version(s) affected: 5.2

Description
I have symfony/mailer and symfony/mailjet-mailer installed.

My composer.json contains this.

{
#...
        "symfony/mailer": "5.2.*",
        "symfony/mailjet-mailer": "^5.2@dev",
#...
}

My .env contains:

MAILER_DSN=mailjet+smtp://xxx:yyy@default

I get the following error:

The "mailjet+smtp" scheme is not supported.

I also get a similar error with mailjet+api

@nicolas-grekas
Copy link
Member

Hello,
why do you think this should work? Did you read it should somewhere?

@stof
Copy link
Member

stof commented Sep 28, 2020

can you give the stack trace of the error ?

@stof
Copy link
Member

stof commented Sep 28, 2020

I think the issue is that mailjet-mailer is not know by the FrameworkBundle DI extension:

$classToServices = [
SesTransportFactory::class => 'mailer.transport_factory.amazon',
GmailTransportFactory::class => 'mailer.transport_factory.gmail',
MandrillTransportFactory::class => 'mailer.transport_factory.mailchimp',
MailgunTransportFactory::class => 'mailer.transport_factory.mailgun',
PostmarkTransportFactory::class => 'mailer.transport_factory.postmark',
SendgridTransportFactory::class => 'mailer.transport_factory.sendgrid',
SendinblueTransportFactory::class => 'mailer.transport_factory.sendinblue',
];

@stof stof added Good first issue Ideal for your first contribution! (some Symfony experience may be required) Help wanted Issues and PRs which are looking for volunteers to complete them. labels Sep 28, 2020
@xabbuh xabbuh added the Mailer label Sep 28, 2020
@sandoche
Copy link
Author

sandoche commented Sep 28, 2020

Hello,
why do you think this should work? Did you read it should somewhere?

Saw it here:
https://symfony.com/doc/master/mailer.html (5.2 version)

I'll send the stack trace later today.
I did checkout my code.

@fabpot fabpot closed this as completed Sep 30, 2020
fabpot added a commit that referenced this issue Sep 30, 2020
This PR was merged into the 5.1 branch.

Discussion
----------

[FrameworkBundle] Add Mailjet definition

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

Fixes #38316

Commits
-------

132bd2a [FrameworkBundle] Add Mailjet definition
@vico-gs
Copy link

vico-gs commented Jan 22, 2021

@sandoche , hey I've had the exact same situation this afternoon, turns out the package 'symfony/framework-bundle' supports this update if you use ^5.1 at least. Had to update some of packages as well as many conflicts follows his upgrade .

@7system7
Copy link

I have the same issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Good first issue Ideal for your first contribution! (some Symfony experience may be required) Help wanted Issues and PRs which are looking for volunteers to complete them. Mailer Status: Needs Review
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants