magento / zf1

ZF1 adopted for Magento 2. Use composer in Magento 2 root to download it.
BSD 3-Clause "New" or "Revised" License
60 stars 61 forks source link

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

Closed lfluvisotto closed 6 years ago

lfluvisotto commented 6 years ago

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 commented 6 years ago

CLA assistant check
All committers have signed the CLA.

lfluvisotto commented 6 years ago

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

https://github.com/magento/magento2/issues/6146

lfluvisotto commented 6 years ago

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

buskamuza commented 6 years ago

Zend Mail from ZF1 is not used in Magento 2 and may be removed soon as last usages will be removed from Magento https://github.com/magento-engcom/php-7.2-support/pull/136

Based on what's described here https://github.com/magento/magento2/issues/6146#issuecomment-375937686 , it's an issue with ZF2's zend-mail.

elvinristi commented 6 years ago

@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.