Skip to content

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

Closed
aiirvizionz wants to merge 2 commits into
profullstack:masterfrom
aiirvizionz:clientkit/public-keys-batch-guard
Closed

fix(crypto): bound public key batch lookups#230
aiirvizionz wants to merge 2 commits into
profullstack:masterfrom
aiirvizionz:clientkit/public-keys-batch-guard

Conversation

@aiirvizionz

Copy link
Copy Markdown
Contributor

Summary

  • cap authenticated public-key batch requests at 100 IDs
  • reject non-string and blank IDs
  • trim and deduplicate IDs before service-role lookups

Fixes #228.

Validation

  • git diff --check
  • Targeted Vitest coverage added, but the local runner fails during config loading because @vitejs/plugin-react imports the non-exported vite/internal subpath. No test cases executed due to that pre-existing dependency mismatch.

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

2 participants