Skip to content

fix: distinguish unreachable backend from expired authentication - #301

Merged
jacob-bd merged 1 commit into
mainfrom
codex/fix-transient-backend-auth
Aug 19, 2026
Merged

fix: distinguish unreachable backend from expired authentication#301
jacob-bd merged 1 commit into
mainfrom
codex/fix-transient-backend-auth

Conversation

@jacob-bd

@jacob-bd jacob-bd commented Aug 19, 2026

Copy link
Copy Markdown
Owner

Summary

  • Distinguish transport, timeout, DNS, and HTTP 5xx failures from expired authentication during session refresh.
  • Preserve explicit Google login redirects and expiry messages as authentication failures.
  • Expose transient backend failures through the existing NLMError hierarchy so the CLI renders a friendly network error instead of a traceback.
  • Add regression tests for classification, auth recovery, and CLI output.

This supersedes the contributor-fork implementation in #300 while preserving its intended behavior with stronger coverage.

Verification

  • Affected tests: 57 passed
  • Ruff and formatting checks passed
  • Full non-E2E suite: 1424 passed, 38 skipped, 1 deselected

@jacob-bd
jacob-bd merged commit 062a6c9 into main Aug 19, 2026
3 checks passed
@jacob-bd
jacob-bd deleted the codex/fix-transient-backend-auth branch August 19, 2026 20:46
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