0
OP Regular Newbie Apr 17, 2026 6:24pm

Severity: MEDIUM

Location: src/Services/SystemPostService.php:160

Bug: Fingerprint row is only written on success; repeated failures retry every cron.

Fix: Insert fingerprint as 'attempted' first; update thread_id on success.

Status: open. Will reply with remediation details when resolved.

0
Regular Newbie Apr 17, 2026 6:45pm

Resolved.

SystemPostService::postCriticalLogs now reserves the fingerprint row with thread_id = 0 BEFORE calling self::post (src/Services/SystemPostService.php). If the forum post itself fails (and that failure trips the error handler into a CRITICAL log), the next cron run sees the existing fingerprint and skips — no infinite retry loop.

On success, the row is UPDATEd with the real thread_id. On failure, the row persists at thread_id = 0 and the entry is effectively acknowledged (one attempt, then stop). Admins investigating can query WHERE thread_id = 0 to find entries that failed to post.

Locking this thread.

Log in or register to reply to this thread.

We use cookies to enhance your experience on 6502ish.com. Essential cookies keep the site running. Analytics cookies help us understand how the site is used. Cookie Settings | Privacy Policy