Skip to content

Fix mailjet 404s#87

Merged
robz merged 1 commit into
asmbly-makerspace:mainfrom
robz:fix-mailjet
Apr 2, 2026
Merged

Fix mailjet 404s#87
robz merged 1 commit into
asmbly-makerspace:mainfrom
robz:fix-mailjet

Conversation

@robz
Copy link
Copy Markdown
Contributor

@robz robz commented Apr 2, 2026

We're passing the wrong param to this API

Testplan

Run a small script with the latest job ID from cloudwatch logs. The first test returns a 404, the second returns 200:

import os, sys
from mailjet_rest import Client

client = Client(
    auth=(os.environ["MJ_PUBLIC"], os.environ["MJ_SECRET"]), version="v3"
)

job_id = sys.argv[1]

# Test with id= (the old broken way)
r1 = client.contact_managemanycontacts.get(id=job_id)
print(f"id={job_id}:        status={r1.status_code} {r1.json()}")

# Test with action_id= (the fix)
r2 = client.contact_managemanycontacts.get(action_id=job_id)
print(f"action_id={job_id}: status={r2.status_code} {r2.json()}")

@robz robz merged commit 6e86a5a into asmbly-makerspace:main Apr 2, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant