Skip to content

fix(api): validate property run limits#19

Merged
ralyodio merged 1 commit into
profullstack:masterfrom
rissrice2105-agent:fix/runs-limit-validation
Jun 18, 2026
Merged

fix(api): validate property run limits#19
ralyodio merged 1 commit into
profullstack:masterfrom
rissrice2105-agent:fix/runs-limit-validation

Conversation

@rissrice2105-agent

Copy link
Copy Markdown
Contributor

Summary

  • validate property run limit query values as positive integers
  • default malformed, fractional, zero, or negative limits to 25
  • keep the existing 100 item cap for valid positive limits
  • add regression coverage for invalid and oversized limits

Closes #18

Tests

  • corepack pnpm install --ignore-scripts
  • corepack pnpm --filter @profullstack/threatcrush-web test -- "src/app/api/orgs/[id]/properties/[property_id]/runs/tests/route.test.ts"
  • corepack pnpm --filter @profullstack/threatcrush-web build

@ralyodio ralyodio merged commit 5b04712 into profullstack:master Jun 18, 2026
9 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.

Property runs API accepts negative limit values

2 participants