-
Notifications
You must be signed in to change notification settings - Fork 30
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
Cannot send test email #62
Comments
Tagging @sunilpawar |
@mattwire can you confirm the sending domain (from email address)? |
@sunilpawar Sorry :-) FROM and TO are both valid email addresses - I replaced them with XXX and YYY for this ticket. |
@mattwire its working on our environment |
@sunilpawar Sure, everything works perfectly with the previous commit: b5b04fb I've tested on:
Both have the same issue. |
@nganivet look like curl re-use connection not working with php 7. lets discus this. |
@mattwire Thanks for reporting. Can you try adding curl_setopt($ch, CURLOPT_CUSTOMREQUEST, "POST"); just before line 113 of CRM/Sparkpost.php and test to see if it fixes your issue? |
@mattwire Might need 'GET' instead of 'POST' in the above, not quite sure as the documentation is lacking. Can you try both and report back here? Thanks. |
@nganivet I replaced the existing line: and that seems to fix the problem. Note that I tested with POST and that failed with a message 404 on creating webhook when trying to send a test email. |
@mattwire OK, we will fix the issue and release a new version. Thanks for your feedback. |
@mattwire Can you please test the latest commit and confirm it works on your side. Was a bit difficult to fix cleanly as documentation on CURLOPT_CUSTOMREQUEST is lacking and this is what caused the issue. |
Using the latest dev it was not working and seemed to match what @mattwire describes here:
I used the latest official release and patched the line suggested here and then the test email is reported as sending correctly (although I haven't received it yet). |
With this fix I get
But changing line 44 to |
why is this issue closed when it clearly is still an issue and the 1.2 version does not work? |
@lsmithgo github automatically closes them when you reference an issue in a commit message |
@nganivet @sunilpawar So the original one-liner suggestion |
@mattwire I changed CURLOPT_CUSTOMREQUEST value and sent test mail. Following are the result for different value. For Failed Result, following is the message. (branch: master with latest changes). |
@sunilpawar I didn't try the web-hook as I switched back to an older version on live sites until the issue with CUSTOMREQUEST is fully resolved. |
Same issue here, I tried the same fix that Matt tried but nothing happens. I revered back to ver 1.1. |
I am getting the same error as @mattwire after I upgraded to CiviCRM 5.4.1 and v1.2 of this extension. I am using php 5.6.37 on Ubuntu 16.04. |
Interestingly, just after I posted this comment I got a delayed receipt from our CiviCRM installation. I tried a test transaction first, didn't get a receipt immediately, then went to the sparkpost configuration to test it. I believe it is working OK on production tasks, but throwing this error only during the "test" function. |
Drupal 7.60, Civi 5.3.2, PHP 5.6.36, getting same error as others (below).
|
@rick02840 I use the extension based on this commit: b5b04fb |
Thanks I think that fixed it! The test email working now here: /civicrm/admin/setting/sparkpost?reset=1 ...will see if membership reminder emails start flowing again. |
Not sure if mail sending is working yet, but I am still seeing these errors below in Drupal logs (/admin/reports/dblog):
UPDATE: UPDATE: |
@mattwire @rick02840 I have Tested this on our dev environment and its working for us. If this work for you, then i will push these changes to repository. Additional Details: |
That patch still doesn't work for me. I can only make v1.2 work by backing out the curl optimizations commit as others have reported here. php 7.0.32 |
This commit 85a4a89 breaks sending test emails for me. I didn't test whether it also breaks sending other email but I'm guessing it does.
When pressing "Save and Send Test Email" from: Administer->System Settings->Outbound Email (Sparkpost)
I get the following error:
The text was updated successfully, but these errors were encountered: