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

From address empty when the plugin is enabled. #116

Closed
mariamghalleb opened this issue Jun 29, 2018 · 7 comments
Closed

From address empty when the plugin is enabled. #116

mariamghalleb opened this issue Jun 29, 2018 · 7 comments

Comments

@mariamghalleb
Copy link

Magento version #: 2.2.4

Edition (EE, CE, OS, etc): CE

Expected behavior: From Address Email shouldn't be empty.

Actual behavior: From Address Email is empty.

Steps to reproduce:

  1. Enable your smtp module
  2. Sales->Orders->Select any order->Write down a comment->Notify customer By Email->Submit Email
    3.The email is sent with no "From address" (see attachement)
    capture

Preconditions

Php 7, Magento CE 2.2.4

PLEASE NOTE:

This occurs only when your SMTP module is enabled.

@srenon
Copy link
Contributor

srenon commented Jun 29, 2018

@mariamghalleb

I'm not able to reproduce your issue on a fresh install of Magento 2.2.4

image

@mariamghalleb
Copy link
Author

can you please screenshot your SMTP configuration?

@palo
Copy link

palo commented Jul 3, 2018

Same here.

  host gmail-smtp-in.l.google.com [***.***.**.**]
    SMTP error from remote mail server after pipelined end of data:
    550-5.7.1 [**.**.***.**      11] Our system has detected that this message is
    550-5.7.1 not RFC 5322 compliant:
    550-5.7.1 'From' header is missing.

Running Magento CE 2.2.4 | PHP 7.0.30 | SMTP pro v1.5.2

Was working fine, not sure which magento update broke it.
Have tried multiple settings.
Can sent correct test email.
Magento order emails are missing 'From' header.
Password recovery is ok.
Nothing relevant in logs.

@palo
Copy link

palo commented Jul 3, 2018

@mariamghalleb quick workaround if your not using multistore:

in 'vendor/magepal/magento2-gmailsmtpapp/Plugin/Mail/TransportPlugin.php' at line 70 add
$message->setFrom('yourshop@example.com');

Example:

        if ($returnPathEmail !== null && $dataHelper->getConfigSetFrom()) {
            $message->clearFrom();
            $message->setFrom($returnPathEmail);
        }
        
        $message->setFrom('yourshop@example.com');

recompile, clear cache, ...

@srenon
Copy link
Contributor

srenon commented Jul 3, 2018

@palo ... you should update to the latest version

@srenon
Copy link
Contributor

srenon commented Jul 3, 2018

@mariamghalleb @palo ... take a look at #113

@palo
Copy link

palo commented Jul 3, 2018

@srenon thanks for helping

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

No branches or pull requests

3 participants