Skip to content

Remove old Zapier IP address#1056

Merged
mkllnk merged 1 commit into
openfoodfoundation:masterfrom
mkllnk:remove-old-zapier-ip
Apr 28, 2026
Merged

Remove old Zapier IP address#1056
mkllnk merged 1 commit into
openfoodfoundation:masterfrom
mkllnk:remove-old-zapier-ip

Conversation

@mkllnk

@mkllnk mkllnk commented Apr 21, 2026

Copy link
Copy Markdown
Member

Zapier updated their IP addresses. The official entries are listed in this article:

The old IP address should not be in use anymore. We should test that though.

@mkllnk mkllnk self-assigned this Apr 21, 2026
@github-project-automation github-project-automation Bot moved this to All the things 💤 in OFN Delivery board Apr 21, 2026
@mkllnk mkllnk requested a review from rioug April 21, 2026 04:08
@mkllnk mkllnk moved this from All the things 💤 to Code review 🔎 in OFN Delivery board Apr 21, 2026

@dacook dacook left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm sure that we're not using it in Australia. It might be worth checking with Bethan also?

@rioug

rioug commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

I'm sure that we're not using it in Australia. It might be worth checking with Bethan also?

It was flagged by Karla from OFN Canada, so we should check with them as well.

@mkllnk

mkllnk commented Apr 21, 2026

Copy link
Copy Markdown
Member Author

I'm sure that we're not using it in Australia.

I just logged into our account and it looks like there are a few enterprises set up to add new customers to Mailchimp via Zapier. I haven't checked if those enterprises are still trading though and half of the zaps seem to be broken.

Update: I think that we are checking on new enterprise creation. It looks like we have two active zaps accessing Postgres. We could probably replace them with n8n but I'm not sure that there's a need for that at the moment.

@dacook dacook moved this from Code review 🔎 to In Progress ⚙ in OFN Delivery board Apr 27, 2026
@dacook

dacook commented Apr 27, 2026

Copy link
Copy Markdown
Member

As this is still being used, I'm guessing we will need to update the IP address before they close down the old ones?

@rioug

rioug commented Apr 27, 2026

Copy link
Copy Markdown
Contributor

It's done already : #1029 , I think we just need to make sure that automation are still working with just the new ip/

@dacook

dacook commented Apr 27, 2026

Copy link
Copy Markdown
Member

Oh ok. Your PR says they are due by Feb 2026, so I would consider the test complete ;)
In which case, we can merge and provision, no?

@dacook dacook moved this from In Progress ⚙ to Ready to go 🚀 in OFN Delivery board Apr 27, 2026
@mkllnk

mkllnk commented Apr 28, 2026

Copy link
Copy Markdown
Member Author

we can merge and provision, no?

Yes, I think so. Production test should be enough.

@mkllnk mkllnk merged commit dc86731 into openfoodfoundation:master Apr 28, 2026
2 checks passed
@github-project-automation github-project-automation Bot moved this from Ready to go 🚀 to Done in OFN Delivery board Apr 28, 2026
@mkllnk mkllnk deleted the remove-old-zapier-ip branch April 28, 2026 03:25
@mkllnk

mkllnk commented Apr 28, 2026

Copy link
Copy Markdown
Member Author

Provisioning:

ansible-playbook --limit au_staging playbooks/provision.yml -t dbserver
  • au-staging
  • fr
  • ca
  • us
  • au
  • uk

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Remove old zapier ip address

3 participants