fix: small fixes in analytics_job #130

New Issue

AtHeartEngineer · 2025-07-08T08:40:38-04:00

AtHeartEngineer commented

2025-07-08 08:40:38 -04:00

Originally created by @ItalyPaleAle on 5/28/2025

Ensure there's a timeout in the HTTP request (using a context with timeout). The HTTP client passed doesn't have a timeout, so in certain cases the operation could block forever.
Skip registering the job entirely if analytics are disabled
Add a retry (with backoff) to handle transient failures

*Originally created by @ItalyPaleAle on 5/28/2025* 1. Ensure there's a timeout in the HTTP request (using a context with timeout). The HTTP client passed doesn't have a timeout, so in certain cases the operation could block forever. 2. Skip registering the job entirely if analytics are disabled 3. Add a retry (with backoff) to handle transient failures

AtHeartEngineer closed this issue

2025-07-08 08:40:38 -04:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github/pocket-id#130