Skip to content

Handle oversized base62 values#65

Merged
svix-jbrown merged 1 commit into
svix:mainfrom
gaoflow:fix-base62-overflow-validation
Jun 22, 2026
Merged

Handle oversized base62 values#65
svix-jbrown merged 1 commit into
svix:mainfrom
gaoflow:fix-base62-overflow-validation

Conversation

@gaoflow

@gaoflow gaoflow commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • convert oversized base62 decoded values from a raw OverflowError into ByteArrayLengthException
  • add coverage for 27-character base62 values that exceed the 20-byte KSUID range for both Ksuid and KsuidMs

Tests

  • uv run pytest
  • uv run ./scripts/lint.sh
  • uv run ruff check ksuid tests
  • uv run ruff format --check ksuid tests
  • python -m compileall -q ksuid tests
  • git diff --check

@gaoflow gaoflow requested a review from a team as a code owner June 19, 2026 03:17
@svix-jbrown

Copy link
Copy Markdown
Contributor

Thanks for the contribution

@svix-jbrown svix-jbrown merged commit 3ee701d into svix:main Jun 22, 2026
12 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.

2 participants