- π©πͺGermany szeidler Berlin
I agree. It's still an issue in 2.x-dev. We're getting a lot of those "errors" in our logs, which creates a lot of noise.
In this case exposing it as an error makes sense from the Mailchimp API perspective, but for the Drupal module it's not more than a warning and should be logged like this.
- Merge request !87Issue #3186936 by szeidler: Do not log Mailchimp subscribe validation errors as Drupal errors β (Merged) created by szeidler
- last update
5 months ago 17 pass - Status changed to Needs review
5 months ago 5:19pm 18 June 2024 - π©πͺGermany szeidler Berlin
According to the Mailchimp API documentation https://mailchimp.com/developer/marketing/docs/errors/ all "Invalid Resource" on subscribe should be validation issues on the Mailchimp end. The MR here prevents logging those into the Drupal logs at all.
- Status changed to Needs work
4 months ago 3:17pm 25 July 2024 - πΊπΈUnited States xenophyle
Do you think you could add a setting to make this non-logging optional?
- π©πͺGermany szeidler Berlin
No, I don't think it should be an option.
This is an error the Mailchimp API returns, that is in my opinion not an error being relevant for the Drupal site and therefore should not be logged as an error. Drupal errors should be possible to resolve from the site owner. But this validation is kind of spam-protection on the Mailchimp-side we cannot influence. A compromise that I can see is that we log the `Invalid Resource` responses with a different severity, for example warning or info.
What do you think?
- πΊπΈUnited States xenophyle
Yes, we can definitely reduce the severity.
As far as reducing the frequency, are you using any kind of captcha? - π©πͺGermany szeidler Berlin
Any kind of Drupal form spam protection can mitigate the issue, yes. In case for example the captcha catches the one that Mailchimp identifies as Spam.
- Status changed to Needs review
4 months ago 7:47am 2 August 2024 - π©πͺGermany szeidler Berlin
@xenophyle I updated the MR to log the Mailchimp validation errors with a lower severity (notice).
-
xenophyle β
committed a9164fa0 on 2.x authored by
szeidler β
Issue #3186936 by szeidler, ikit-claw: Should the message failed to...
-
xenophyle β
committed a9164fa0 on 2.x authored by
szeidler β
- Status changed to Fixed
4 months ago 3:54pm 4 August 2024 Automatically closed - issue fixed for 2 weeks with no activity.