Skip to content

Validate message load limits#82

Merged
ralyodio merged 1 commit into
profullstack:masterfrom
phucnguyen1707:validate-message-load-limit
Jun 11, 2026
Merged

Validate message load limits#82
ralyodio merged 1 commit into
profullstack:masterfrom
phucnguyen1707:validate-message-load-limit

Conversation

@phucnguyen1707

Copy link
Copy Markdown
Contributor

Fixes #81.

Summary

  • Validate POST /api/messages/load limit values before database work
  • Only pass normalized integer limits from 1 to 100 to Supabase .limit(...)
  • Use the normalized limit for hasMore
  • Add a route unit test covering invalid limits before DB access

Tests

  • pnpm vitest run src/app/api/messages/load/route.test.js
  • pnpm vitest run src/app/api/messages/send/route.test.js src/app/api/messages/load/route.test.js

@ralyodio ralyodio merged commit cba49e7 into profullstack:master Jun 11, 2026
8 checks passed
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.

Validate message load limit before querying

2 participants