Skip to content

fix(importer): defer processing when no NNTP providers are configured#355

Merged
javi11 merged 1 commit intomainfrom
fix/61-no-providers-check
Feb 26, 2026
Merged

fix(importer): defer processing when no NNTP providers are configured#355
javi11 merged 1 commit intomainfrom
fix/61-no-providers-check

Conversation

@javi11
Copy link
Copy Markdown
Owner

@javi11 javi11 commented Feb 26, 2026

Summary

  • Before starting NZB segment downloads, check poolManager.HasPool()
  • If no providers are configured, return a retryable (non-fatal) error so the item stays in the queue
  • STRM files are excluded from this check — they are served over HTTP and don't need an NNTP connection

Test plan

  • Build passes: go build ./...
  • Without any providers configured, queue items stay in a pending state instead of failing permanently
  • After adding a provider, the previously-deferred items resume processing

Fixes #61

🤖 Generated with Claude Code

Before attempting NZB segment downloads, check whether the NNTP pool
has at least one provider. If not, return a retryable error so the item
stays in the queue instead of failing permanently. STRM files are
unaffected since they are served via HTTP and don't need an NNTP pool.

Fixes #61

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@javi11 javi11 merged commit 2f53fdf into main Feb 26, 2026
1 check passed
@javi11 javi11 deleted the fix/61-no-providers-check branch February 26, 2026 14:07
drondeseries referenced this pull request in drondeseries/altmount_old Apr 16, 2026
Before attempting NZB segment downloads, check whether the NNTP pool
has at least one provider. If not, return a retryable error so the item
stays in the queue instead of failing permanently. STRM files are
unaffected since they are served via HTTP and don't need an NNTP pool.

Fixes #61

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

[FEAT] Prevent download attempts when no providers are active

1 participant