Skip to content

Attempt to fix #43 by retrying on likely race condition 404s#72

Merged
robz merged 1 commit into
asmbly-makerspace:mainfrom
epowell:epowell/mailjet-retries
Mar 16, 2026
Merged

Attempt to fix #43 by retrying on likely race condition 404s#72
robz merged 1 commit into
asmbly-makerspace:mainfrom
epowell:epowell/mailjet-retries

Conversation

@epowell
Copy link
Copy Markdown
Contributor

@epowell epowell commented Feb 6, 2026

  1. Added tenacity to requirements.txt
  2. Updated get_job_status method with retry logic:
    5 retry attempts (stop_after_attempt(5))
    Exponential backoff starting at 2 seconds, max 10 seconds (wait_exponential(multiplier=1, min=2, max=10))

When a 404 is encountered, it now returns "Processing" instead of logging an error, which triggers the retry mechanism

This handles the potential case where the job hasn't been created yet in Mailjet's system, or status is not yet available.

@robz robz merged commit 394fb83 into asmbly-makerspace:main Mar 16, 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.

2 participants