Repository navigation
Etsy: say that shop reviews refuse large offsets - #1503
Merged
Merged
Conversation
helpcodeai
enabled auto-merge (squash)
November 4, 2026 08:14
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
etsy_get_shop_reviewshad no bounds and no note onlimit/offset. Etsy treats the reviews endpoint as public and refuses large offsets: Cloud logged "Offset exceeds the maximum allowed for anonymous requests" 6 times in one workspace on 3-4 Nov. Same fix as #1471 for shop section listings.Changes
limit: 1 to 100 (default 25),offset: at least 0, with the "keep it small" note.min_createdto reach older reviews instead of paging far.Only
packages/backend/src/adapters/intl/etsy.jsonchanges;catalog.tsregenerates without a diff. Cloud and self-hosted behave the same; installed connectors get the new schema after "Apply update" on the connector page.Type
Testing
From the repo root:
node scripts/validate-adapters.mjs --warn→ 709 passed, 0 failed, no new Etsy warningnode scripts/regenerate-catalog.mjs→ no diffIn
packages/backend:npx jest src/adapters/catalog.spec.ts src/adapters/intl/etsy --maxWorkers=4→ 14,753 passedChecklist
npm test)Related Issues
Follow-up to #1471.