Skip to content

Add safety cap to ComicVine issue pagination#4

Closed
dotMavriQ wants to merge 1 commit into
mainfrom
fix/comicvine-pagination-cap
Closed

Add safety cap to ComicVine issue pagination#4
dotMavriQ wants to merge 1 commit into
mainfrom
fix/comicvine-pagination-cap

Conversation

@dotMavriQ
Copy link
Copy Markdown
Owner

Summary

  • Caps ComicVine volume issue fetching at 10 pages (1000 issues) to prevent request timeouts on very large series

Test plan

  • Import a large comic series (100+ issues) and verify it completes without timeout
  • Verify the 1000 issue cap doesn't truncate normal-sized series

@coderabbitai full review

Limits volume issue fetching to 10 pages (1000 issues) to prevent
timeouts on very large series.
@dotMavriQ
Copy link
Copy Markdown
Owner Author

Landed on main directly in 4e46572 -- branch had merge conflicts from the logging changes.

@dotMavriQ dotMavriQ closed this Mar 14, 2026
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