Skip to content

fix: Say that the data system retries after an unrecoverable HTTP error - #501

Open
beekld wants to merge 1 commit into
bklimt/SDK-3425/synchronizer-off-resultfrom
bklimt/SDK-3425/unrecoverable-error-log
Open

beekld wants to merge 1 commit into
bklimt/SDK-3425/synchronizer-off-resultfrom
bklimt/SDK-3425/unrecoverable-error-log

Conversation

@beekld

@beekld beekld commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

Summary

#489 changed the ERROR log for an unrecoverable HTTP status, such as a 401, from "giving up permanently" to "will not retry until restarted." The synchronizer does stop, but the data system parks it and starts it again after a delay. An operator could read "restarted" as "restart the process," which also resets the park backoff. The message now says that the data system will retry the synchronizer after a delay.


Note

Overview
Updates the ERROR log emitted when an HTTP status is treated as unrecoverable by the synchronizer (e.g. most 4xx besides 400/408/429).

The message no longer says the error will not retry until restarted, which could be read as needing a process restart. It now states that the data system will retry it after a delay, matching how the synchronizer is parked and restarted by the data system rather than giving up permanently.

Reviewed by Cursor Bugbot for commit 7674d5a. Bugbot is set up for automated code reviews on this repo. Configure here.

@beekld
beekld added this pull request to stack #503 November 2, 2026 21:04
@beekld
beekld marked this pull request as ready for review November 2, 2026 21:05
@beekld
beekld requested a review from a team as a code owner November 2, 2026 21:05
@beekld
beekld force-pushed the bklimt/SDK-3425/unrecoverable-error-log branch from 7812ed4 to 44e243b Compare November 2, 2026 21:31
@beekld
beekld force-pushed the bklimt/SDK-3425/unrecoverable-error-log branch from 44e243b to 4fda77d Compare November 2, 2026 21:59
@beekld
beekld force-pushed the bklimt/SDK-3425/unrecoverable-error-log branch from 4fda77d to 7674d5a Compare November 3, 2026 00:05
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