Skip to content

fix(search): validate pagination params strictly#127

Open
rissrice2105-agent wants to merge 1 commit into
profullstack:masterfrom
rissrice2105-agent:fix/search-pagination-parsing
Open

fix(search): validate pagination params strictly#127
rissrice2105-agent wants to merge 1 commit into
profullstack:masterfrom
rissrice2105-agent:fix/search-pagination-parsing

Conversation

@rissrice2105-agent

Copy link
Copy Markdown
Contributor

Summary

  • add a shared strict integer parser for API pagination params
  • reject partial values like 10abc in public search/browse endpoints
  • preserve existing status codes and validation messages for invalid pagination inputs

Tests

  • pnpm exec vitest run src/lib/api/pagination.test.ts
  • pnpm exec tsc --noEmit
  • git diff --check

Notes

This is a small bug fix for the pinned repos Ugig flow. No invoice will be sent until the PR is merged/accepted.

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