search

cividesk / com.cividesk.email.sparkpost

This extension allows CiviCRM to send emails and process bounces through the SparkPost service.
10 stars 30 forks source link

Emails recorded as bounced when running a stress test #17

Closed madurak closed 8 years ago

madurak commented 8 years ago

Hi Nicholas,

We ran a stress test with 30K emails and found that in CiviMail reports most of these are recorded as bounced. However under sparkpost reports these are sent and the emails are also received. Can you take a look at this. I have attached some screenshots, let me know if you need any other information.

Tested in: Civi 4.6.14

screen shot 2016-04-11 at 10 23 54 screen shot 2016-04-11 at 10 25 14

-Madura

laryn commented 8 years ago

I see something similar with much smaller numbers of recipients. My latest micro-mailing went to 25 addresses and 23 show as bounces in CiviCRM (with bounce type of "Syntax"), although in SparkPost no bounces are reported. I am getting responses from folks who are on the "bounced" list, so I know they went through.

nganivet commented 8 years ago

Have you updated to the latest version? You need to do a 'git pull' in the Sparkpost extension's directory. The version 1.0 released last weeks corrects this issue.

------ Original Message ------ From: "Laryn - CEDC.org" notifications@github.com To: "cividesk/com.cividesk.email.sparkpost" com.cividesk.email.sparkpost@noreply.github.com Sent: 4/14/2016 1:23:31 PM Subject: Re: [cividesk/com.cividesk.email.sparkpost] Emails recorded as bounced when running a stress test (#17)

I see something similar with much smaller numbers of recipients. My latest micro-mailing went to 25 addresses and 23 show as bounces in CiviCRM (with bounce type of "Syntax"), although in SparkPost no bounces are reported. I am getting responses from folks who are on the "bounced" list, so I know they went through.

— You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub

laryn commented 8 years ago

Hmm... I thought I had the latest version but I guess I can't say for certain. I will refresh my copy and keep an eye on it when I next have to send a mailing. Thanks!

nganivet commented 8 years ago

Closed as due to outdated version.

ginkgomzd commented 8 years ago

Hi @nganivet, We are seeing something similar. Can you point us to the code that is supposed to address this?

nganivet commented 8 years ago

https://github.com/cividesk/com.cividesk.email.sparkpost/commit/9c393bd176f575aa01f98a3e8c9cb1c5dcf89534

ginkgomzd commented 8 years ago

based on the comment, and my limited interpretation of what that commit is doing... I think that is addressing the reverse problem? Bounces not getting logged? We are seeing bounces logged when they should not be. I suggest re-opening this issue. We will look into resolving it but won't be able to right away.

nganivet commented 8 years ago

What does 'We are seeing bounces logged when they should not be' mean? Shouldn't all bounces get logged? Have you looked into the ConfigandLog directory for the file sparkpost_log? Does it contain any errors relevant to your case?

I will issue version 1.1 over the week-end with many improvements.

------ Original Message ------ From: "Michael Z Daryabeygi" notifications@github.com To: "cividesk/com.cividesk.email.sparkpost" com.cividesk.email.sparkpost@noreply.github.com Cc: "nganivet" nicolas@cividesk.com; "Mention" mention@noreply.github.com Sent: 4/29/2016 1:17:00 PM Subject: Re: [cividesk/com.cividesk.email.sparkpost] Emails recorded as bounced when running a stress test (#17)

based on the comment, and my limited interpretation of what that commit is doing... I think that is addressing the reverse problem? Bounces not getting logged? We are seeing bounces logged when they should not be. I suggest re-opening this issue. We will look into resolving it but won't be able to right away.

— You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub

elisseck commented 8 years ago

I had the same issue when Laryn was commenting - The issue of a large % of mailings bouncing for reason "syntax" was solved when I pulled on 4-20-16 (for civiCRM 4.6.4), if that's what you are seeing.

It might be really obvious, not sure, but we did have to actually uninstall and reinstall the extension completely rather than just pushing a new file before the issue went away for us.

ginkgomzd commented 8 years ago

Same thing as original poster: during a stress-test, emails marked as delayed and then delivered on SparkPost are marked bounced in CiviCRM. Will check the logs.

ginkgomzd commented 8 years ago

We dug deeper and came up with a feature request. See #26