Great module and is working fine except I see that in MailChimp the IP addresses associated with subscriptions made via this module are all of my web server.

Is it possible for the module to instead pass on to MailChimp the IP address of the subscribing user?

This is important for providing proof of opt-in subscription. Otherwise, we will need to keep the webform submission log to retain the IP address with seems to me to be an unnecessary duplication of data.

Thanks.

CommentFileSizeAuthor
#5 send-client-IP-to-mailchimp-1978650-5.patch1.74 KBmdixoncm
#4 send-client-IP-to-mailchimp-1978650-4.patch3.53 KBmdixoncm
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mdixoncm’s picture

mdixoncm CreditAttribution: mdixoncm at ComputerMinds commented
Issue summary: View changes

So - some digging around in the mailchimp API docs - it looks like the value optin_ip needs to be set in the merge_vars that are sent to the subscribe call.

There is a rather cryptic looking line in the Mailchimp docs saying "Abusing this may cause your account to be suspended. We do validate this and it must not be a private IP address." - but with no more information on what abusing the field looks like.

I suggest we add an option to the mailchimp module to send the client IP address, which we default to NO - and then folk can enable the functionality if they are feeling brave :)

mdixoncm’s picture

mdixoncm CreditAttribution: mdixoncm at ComputerMinds commented
Project: Webform Mailchimp » Mailchimp
Version: 7.x-1.0-rc2 » 8.x-1.x-dev
Component: Code » General

I think this is actually something that needs doing against the mailchimp module, rather than the mailchimp webform module - moving project - hope this is OK.

mdixoncm’s picture

mdixoncm CreditAttribution: mdixoncm at ComputerMinds commented
Version: 8.x-1.x-dev » 7.x-3.x-dev
mdixoncm’s picture

mdixoncm CreditAttribution: mdixoncm at ComputerMinds commented
FileSize
send-client-IP-to-mailchimp-1978650-4.patch3.53 KB

First stab at a patch for this.

mdixoncm’s picture

mdixoncm CreditAttribution: mdixoncm at ComputerMinds commented
FileSize
send-client-IP-to-mailchimp-1978650-5.patch1.74 KB

First stab went a bit wonky - some whitespace issues in there - second stab looks a bit cleaner.

ruscoe’s picture

ruscoe CreditAttribution: ruscoe at ThinkShout for Mailchimp commented
Status: Active » Closed (won't fix)

We're going to be ending support for the 7.x-3.x branch due to the upcoming deprecation of the v2.0 MailChimp API. If you're still interested in this feature, please could you recreate your patch for the 7.x-4.x branch?