Skip to content

Fix for > https://github.com/magento/magento2/issues/6146 #15

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

Closed

Conversation

lfluvisotto
Copy link

@lfluvisotto lfluvisotto commented Jun 2, 2018

The additional_parameters parameter can be used to pass additional flags as command line options to the program configured to be used when sending mail, as defined by the sendmail_path configuration setting.

For example, this can be used to set the envelope sender address when using sendmail with the -f sendmail option

@magento-cicd2
Copy link

magento-cicd2 commented Jun 2, 2018

CLA assistant check
All committers have signed the CLA.

@lfluvisotto
Copy link
Author

@magento-engcom-team I think this pull request in zend framework will fix the issue:

magento/magento2#6146

@lfluvisotto
Copy link
Author

@buskamuza @joanhe
Could you have a look on this PR?

@buskamuza
Copy link

Zend Mail from ZF1 is not used in Magento 2 and may be removed soon as last usages will be removed from Magento magento-engcom/php-7.2-support#136

Based on what's described here magento/magento2#6146 (comment) , it's an issue with ZF2's zend-mail.

@buskamuza buskamuza closed this Jun 8, 2018
@elvinristi
Copy link

elvinristi commented Aug 31, 2018

@buskamuza , this is 2.3+ what you are reffering here. No clients from 2.2 will go there in near future.

With this "Close" you are saying that you don't support 2.2 anymore?

You should be reviewing this by Contributor if this fits properly to 2.2 or give workable alternatives. Why is this that Magento UA devs are climbing behind this invisible wall and just without going further details, just closes PRs?

You have here community supporting to make this solution workable and you are just ignoring them. not good.

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

Successfully merging this pull request may close these issues.

4 participants