Skip to content

fix(crypto): bound public key batch lookups - #229

Merged
ralyodio merged 1 commit into
profullstack:masterfrom
rissrice2105-agent:fix/public-key-batch-validation
Aug 3, 2026
Merged

fix(crypto): bound public key batch lookups#229
ralyodio merged 1 commit into
profullstack:masterfrom
rissrice2105-agent:fix/public-key-batch-validation

Conversation

@rissrice2105-agent

Copy link
Copy Markdown
Contributor

Summary

  • cap public-key batch requests at 100 IDs
  • reject malformed JSON, non-string IDs, blank IDs, and oversized values
  • trim and deduplicate IDs before service-role database lookups
  • add regression coverage for oversized, malformed, invalid, and duplicate-filled batches

Fixes #228

Testing

  • vitest run src/app/api/crypto/public-keys/route.test.js (6 tests)
  • targeted ESLint
  • git diff --check

Comment thread src/app/api/crypto/public-keys/route.js Fixed
Comment thread src/app/api/crypto/public-keys/route.js Fixed
Comment thread src/app/api/crypto/public-keys/route.js Fixed
Comment thread src/app/api/crypto/public-keys/route.js Fixed
Comment thread src/app/api/crypto/public-keys/route.js Fixed
@rissrice2105-agent
rissrice2105-agent force-pushed the fix/public-key-batch-validation branch from 59474c3 to ecff434 Compare August 1, 2026 23:25
@ralyodio
ralyodio merged commit 4934bc8 into profullstack:master Aug 3, 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.

Unbounded public-key batch lookup allows query amplification

3 participants