Comments (12)
btw the platform is gonna survive if the people who use it participate, there is no big company paying for everybody here and there's too little participation or support. personally I work my ass off for free in order to still have something to provide to my customers, which, btw, don't pay me for the OM development but just to apply those improvements to their store.
So my suggestion is to submit a PR to aschroeder/smtp_pro because realistically we're not gonna be able to redevelop and merge something like that in the core.
from magento-lts.
why smtppro doesn't work for you? I've have on a lot of stores on v20/php8.3 and works perfectly
from magento-lts.
ah you mean that you can't configure a different smtp for every website? but the emails are correctly sent with the proper "from email address" depending on the store
from magento-lts.
First of all if you check the configuration options scopes, they're set to website = 0 so they won't work for multi-site, but we can patch this.
Secondly, after setting it up it throws this error for TLS on 587
if ($this->_secure == 'tls') {
$this->_send('STARTTLS');
$this->_expect(220, 180);
if (!stream_socket_enable_crypto($this->_socket, true, STREAM_CRYPTO_METHOD_TLS_CLIENT)) {
/**
* @see Zend_Mail_Protocol_Exception
*/
#require_once 'Zend/Mail/Protocol/Exception.php';
throw new Zend_Mail_Protocol_Exception('Unable to connect via TLS');
}
$this->_ehlo($host);
}
And that error for SSL on 465
// open connection
$this->_socket = @stream_socket_client($remote, $errorNum, $errorStr, self::TIMEOUT_CONNECTION);
if ($this->_socket === false) {
if ($errorNum == 0) {
$errorStr = 'Could not open socket';
}
/**
* @see Zend_Mail_Protocol_Exception
*/
#require_once 'Zend/Mail/Protocol/Exception.php';
throw new Zend_Mail_Protocol_Exception($errorStr);
}
Granted the OpenMage version I tested this is low > OpenMage LTS 20.0.7 and using php 7.4 but still
I'm assuming it's not creating the stream context properly in each case causing the failure
from magento-lts.
ah you mean that you can't configure a different smtp for every website? but the emails are correctly sent with the proper "from email address" depending on the store
Regarding that, in most providers if you set a different from address than the defined smtp address it will either get overwritten or the mail will be rejected by amavis. So you need to have the same smtp email address as the one you're sending from.
Sometimes a single smtp connection is not enough so it woudl be a bonus to have multiple smtp connections tied to email addresses.
But in any case I was referring to it having some issues sending as well.
from magento-lts.
I dont know much about the spec, but maybe its possible to not have Magento (PHP) do that but some different process on the host thats "made to do it"? This way we could save mbox (I think thats what they are called) files and that other process handles sending it?
from magento-lts.
I'm seeing https://github.com/aschroder/Magento-SMTP-Pro-Email-Extension/pulls PRs are not processed at all.
Maybe we could fork it and merge those PRs, I don't know, I'm already working on so many things.
from magento-lts.
@fballiano First of all this is not meant to take it personally my friend.
I just see the issues as they arise and while I'm working on a solution I report them here as well. Whether they're bug reports or feature requests or anything I just put them here and see how we can proceed with them. Of course if I had found a solution I'd push it already but I haven't found it yet :)
Also I saw the same on the smtp pro extension, it's pretty much dead. No one cares about it that's why I thought this feature would be important in OpenMage.
Basically every other framework has smtp capabilities out of the box by now using symfony/mailer which is properly supported but because we have Zend ( 👎 ) we're stuck with all the issues it gives
from magento-lts.
All I can say is that all of us should be the change that they want to be (well, this is a quote heheheh), everybody has to take something in his/her own responsibility and do what they need
from magento-lts.
I did not read much of the comments.
But if its a change which does not cause many problems with existing integrations, I dont see a big reason against accepting a PR, if you find/have someone who is going to provide this changes.
from magento-lts.
+1 for a PR that implements it. I would like to contribute. personally I use aschroder smtp that have some issues when email cannot be delivered and block entire queue also some limitation with from headers and multistore under different domains.
this is my fork:
https://github.com/empiricompany/Magento-SMTP-Pro-Email-Extension
from magento-lts.
We are using Advanced SMTP which, I see now, is also in OM module fostering
https://github.com/OpenMageModuleFostering/artsonit_advancedsmtp
/Edit: And Sven Reichel also has a fork of it:
https://github.com/sreichel/magento-ArtsOnIT_AdvancedSmtp
from magento-lts.
Related Issues (20)
- After upgrade to v20.7.0 I got some alert on Admin Login HOT 16
- Fatal error after upgrading from 20.1.1 to 20.8 when clearing cache from Admin Page HOT 2
- Error when saving product with image without description HOT 6
- Problem with Configurable Swatches after PR #3686 HOT 1
- Unable to Override GroupController in OpenMage HOT 2
- N98-magerun tests fails since #2993 HOT 2
- Current category cache issue in navigation block
- UPS REST _parseRestResponse doesn't return rates when only one service is available for a destination
- Saving product from backend causes destroying category product positions in anchor categories.
- All attributes set to be used for search appear in layering navigation when there are no search results HOT 11
- Loopback for the changes Zend directory HOT 5
- The graphs in the Dashboard page no longer display data HOT 10
- Wrong attributes order in comparison window
- Bug on created_at in customer_entity table on update HOT 4
- Deprecated functionality: strlen(): Passing null to parameter #1
- Deprecated functionality: strlen(): Passing null to parameter #1 - Abstract.php HOT 4
- [Minimal Query Length] issue for search strings of 1 - 2 characters HOT 3
- [Backend Navbar] Empty spaces in the classes HOT 5
- New EAV attributes not found without cleaning cache
- Zip-Release 20.10.0 incomplete or too many items? HOT 5
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from magento-lts.